Skip to content

Add GitHub Copilot code review instructions#580

Closed
feliperodri wants to merge 1 commit intomainfrom
review-guidelines
Closed

Add GitHub Copilot code review instructions#580
feliperodri wants to merge 1 commit intomainfrom
review-guidelines

Conversation

@feliperodri
Copy link
Copy Markdown

@feliperodri feliperodri commented Mar 30, 2026

Add .github/copilot-instructions.md to guide GitHub Copilot's automated code reviews for this repository. This will help us triage the coming wave of challenge solutions.

The instructions cover review guidelines for the four types of PRs we receive:

  • Challenge solutions — formal verification rigor (no vacuous proofs, justified assumptions, contracts matching safety docs), Rust std library code quality standards, and success criteria compliance.
  • New challenge proposals — template and tracking issue requirements.
  • New tool applications — CI workflow and book entry requirements.
  • Maintenance — ensuring existing proofs are not weakened.

Also includes a list of common red flags to watch for (e.g., over-constrained assumptions, missing cfg gates, unapproved tools).

Reference: Copilot code review custom instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Signed-off-by: Felipe R. Monteiro <felisous@amazon.com>
@feliperodri feliperodri requested a review from a team as a code owner March 30, 2026 22:26
@feliperodri feliperodri added the Maintenance Maintenance related issues for the challange label Mar 30, 2026
@feliperodri feliperodri enabled auto-merge March 30, 2026 22:27
auto-merge was automatically disabled March 31, 2026 19:57

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance Maintenance related issues for the challange

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants