Repository navigation
Stop asking Copilot to review discovery pull requests - #48
Merged
Merged
Conversation
Copilot never acted on the comment. Its checklist moves to the README's review steps. Assisted-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Sergey Vilgelm <sergey@union.ai>
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are recorded for the removal of Copilot review requests and the move to README-based human review guidance.
No open issues 🎉
What changed in this PR
Removes ineffective Copilot review requests from company discovery pull requests and moves the review checklist into the README for human reviewers.
Changes:
- Removes the review-request comment, its template, and optional token wiring.
- Updates reviewer documentation and removes the obsolete mention exception from agent guidance.
| File | Description |
|---|---|
scripts/discovery-pr.sh |
Removes comment posting and related token and creation-state handling. |
scripts/copilot-discovery-review.md |
Deletes the Copilot review-request template. |
README.md |
Adds company and affiliation validation guidance to human review steps. |
DEVELOPMENT.md |
Removes obsolete Copilot request and token documentation. |
AGENTS.md |
Removes the exception allowing discovery review mentions. |
.github/workflows/sync.yml |
Removes the optional Copilot token environment variable. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The discovery workflow posted a comment asking Copilot to check each company and affiliation proposal, but Copilot has never replied or pushed on any of these pull requests, likely because it does not act on mentions from bots. This removes the comment, its template, and the optional
PULSE_COPILOT_TOKENsecret. The checklist from the comment moves to the README's review steps so people reviewing these pull requests still have it.