Skip to content

Add Copilot review instructions - #41

Merged
cbetta merged 1 commit into
masterfrom
copilot-review-instructions
Oct 23, 2026
Merged

cbetta merged 1 commit into
masterfrom
copilot-review-instructions

Conversation

@cbetta

@cbetta cbetta commented Oct 23, 2026

Copy link
Copy Markdown
Member
Reviewer check
Auth / permissions No
Data access widened No
New secret/env var/key No
CI/deploy/infra No
Dependencies No

Adds Copilot review instructions: report only correctness, security, data-loss and permission problems, plus our security checklist.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The instructions-only change matches the stated purpose, changes no executable behavior, and has no identified blocking issues.

No open issues 🎉

What changed in this PR

Adds repository-level Copilot review guidance focused on actionable correctness and security problems.

Changes:

  • Limits feedback to correctness, security, data-loss, and permission issues.
  • Adds a security checklist covering sensitive data exposure, untrusted model input, account isolation, authorization, and workflow access to secrets.
File Description
.github/​copilot-instructions.md Defines review scope and security checks for Copilot.

Review effort: Balanced

@cbetta
cbetta marked this pull request as ready for review October 23, 2026 15:47
@cbetta
cbetta merged commit cfafe85 into master Oct 23, 2026
9 checks passed
@cbetta
cbetta deleted the copilot-review-instructions branch October 23, 2026 15:47
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.

2 participants