Agent skill: add code-review skill for github copilot that can check for compliance with sundials style guide - #991
Agent skill: add code-review skill for github copilot that can check for compliance with sundials style guide#991balos1 wants to merge 6 commits into
Conversation
…e with sundials style guide
There was a problem hiding this comment.
Pull request overview
Adds a new GitHub Copilot “code-review” skill intended to guide SUNDIALS PR reviews against the project’s developer-guide requirements.
Changes:
- Introduces a
code-reviewskill checklist covering source-code and documentation compliance expectations. - Defines a recommended review comment format for reporting developer-guide violations.
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: balos1 <5669480+balos1@users.noreply.github.com>
Steven-Roberts
left a comment
There was a problem hiding this comment.
This looks reasonable to me. The "Source Review Checklist" and "Documentation Review Checklist" summarize conventions already in the developer docs, so it we update the docs, we'll need to remember to update this. I wonder if a review would differ much if we removed those sections. Copilot is already instructed to review our docs.
Yeah having the checklists enumerated may not be necessary. I am going to remove them. |
No description provided.