-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Update copilot-instructions.md #116049
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update copilot-instructions.md #116049
Conversation
Some tweaks to the instructions to ensure clarity of the agent's reported status
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the copilot-instructions.md file to improve clarity and enforce a stricter standard using MUST/SHOULD notation. Key changes include updated language on code compilation and testing requirements, restructuring of the TOC and references to use local file links, and minor textual clarifications to the instructions.
Tagging subscribers to this area: @dotnet/runtime-infrastructure |
|
||
You MUST follow all code-formatting and naming conventions defined in [`.editorconfig`](/.editorconfig). | ||
|
||
In addition to the rules enforced by `.editorconfig`, you SHOULD: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we also add that we prefer for new tests to be added to existing files rather than to new files as long as there's an existing test file that relevant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can do this in a follow-up change.
Tested on https://github.com/dotnet/runtime/actions/runs/15286139353/job/42996647277