Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 27, 2026

GitHub Copilot agents can mark PRs ready for review without waiting for CI/CD status checks to complete. This creates noise when builds or tests fail.

Changes

Created .github/copilot-instructions.md to enforce pre-review requirements:

  • Build status verification: Azure Pipelines Build and Lint jobs, GitHub Actions CodeQL workflow
  • Test requirements: Unit tests (make unittest) must pass
  • Code quality gates: Linting (make golangci-lint) must pass with no new violations
  • Status check verification: Instructions for using gh pr checks and gh pr view --json statusCheckRollup
  • Exception handling: Guidance for pre-existing failures and documentation requirements
  • Build command reference: Accurate Makefile target descriptions

Key Requirements Enforced

All Copilot PRs must verify:

  • ✅ All Azure Pipelines jobs green
  • ✅ All GitHub Actions workflows green
  • ✅ No failing or pending status checks
  • ✅ Draft status until all checks pass

The instructions file is automatically read by GitHub Copilot when working on PRs in this repository.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 3 commits January 27, 2026 06:51
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
Copilot AI changed the title [WIP] Add instruction for PR readiness based on build status Add Copilot instructions to enforce green status checks before PR review Jan 27, 2026
Copilot AI requested a review from madhanrm January 27, 2026 06:54
@madhanrm madhanrm marked this pull request as ready for review January 27, 2026 07:03
@madhanrm
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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