Skip to content

docs: refresh Copilot instructions - #1016

Merged
Sun Haoran (haoranpb) merged 1 commit into
mainfrom
docs/refresh-copilot-instructions
Nov 9, 2026
Merged

Sun Haoran (haoranpb) merged 1 commit into
mainfrom
docs/refresh-copilot-instructions

Conversation

Copilot AI commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Keep Copilot instructions current by replacing category and harness specifics with general descriptions.

Clarify that PR descriptions should focus on the goal and relevant design, and local development does not need to wait for CI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@haoranpb
Sun Haoran (haoranpb) marked this pull request as ready for review November 9, 2026 09:47
Copilot AI balanced review requested due to automatic review settings November 9, 2026 09:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

There are no active review issues for this documentation-only update to the repository’s Copilot guidance.

No open issues 🎉

What changed in this PR

Refreshes repository-wide Copilot guidance to describe BC-Bench without tying it to specific harnesses or a fixed set of categories.

Changes:

  • Replaces harness-specific and category-specific descriptions with general definitions.
  • Adds guidance for concise PR descriptions and running local checks without waiting for CI.
File Description
.github/​copilot-instructions.md Generalizes benchmark descriptions and clarifies the development workflow; no runtime changes.

Review effort: Balanced

@haoranpb
Sun Haoran (haoranpb) merged commit 2b281bd into main Nov 9, 2026
6 checks passed
@haoranpb
Sun Haoran (haoranpb) deleted the docs/refresh-copilot-instructions branch November 9, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants