Skip to content

chore: configure Copilot task and review guardrails - #9

Merged
rrahimi-uci merged 1 commit into
mainfrom
chore/copilot-task-validation
Aug 17, 2026
Merged

chore: configure Copilot task and review guardrails#9
rrahimi-uci merged 1 commit into
mainfrom
chore/copilot-task-validation

Conversation

@rrahimi-uci

Copy link
Copy Markdown
Owner

Summary

Adds repository-wide Copilot instructions and a pull request template so assigned feature, bug-fix, and development tasks require tests or an explicit non-code validation rationale before human review. Copilot is explicitly prohibited from approving or merging pull requests.

Tests and validation

  • Added repository-specific validation guidance and required PR evidence.
  • Ran git diff --check successfully in each repository.
  • Existing repository CI workflows were inspected and referenced; no existing workflow was removed or weakened.
  • No production code, credentials, or runtime configuration was changed.

Validation commands and results:

  • git diff --check — passed in all 10 repositories.
  • Repository CI — will run on each PR; the instruction files identify the applicable commands for each repository.

Risks and follow-up

The instruction and template are guidance and evidence requirements; Copilot behavior remains subject to GitHub Copilot service availability and the repository CI configuration. Repositories without existing CI retain explicit syntax/documentation validation guidance rather than claiming nonexistent tests. Automatic Copilot review is configured separately at the repository settings level and is comment-only.

Copilot handoff

  • This PR is ready for human review only after applicable validation passes.
  • Copilot did not approve or merge this PR.
  • Merge authority remains with the repository owner.

Copilot AI lite review requested due to automatic review settings August 17, 2026 03:16

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.

Pull request overview

This PR adds repository-level contribution guardrails by introducing a standard pull request template and GitHub Copilot repository instructions, aiming to ensure changes are accompanied by appropriate validation evidence and to explicitly prevent Copilot from taking merge/approval actions.

Changes:

  • Added a pull request template that requires authors to document tests/validation, git diff --check, and risks before requesting human review.
  • Added .github/copilot-instructions.md to codify expected validation steps and prohibit Copilot from approving/merging PRs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/PULL_REQUEST_TEMPLATE.md Adds an evidence-driven PR template (tests/validation, risks, Copilot handoff).
.github/copilot-instructions.md Adds repository-wide Copilot guidance, including explicit “no approve/merge” and validation command expectations.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@rrahimi-uci
rrahimi-uci merged commit 31b91c6 into main Aug 17, 2026
6 of 7 checks passed
@rrahimi-uci
rrahimi-uci deleted the chore/copilot-task-validation branch August 17, 2026 03:40
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