Skip to content

DEVOPS-1052: Add PR comment warning on force push to prompt Copilot re-review#180

Open
RomFloreani wants to merge 1 commit into
mainfrom
DEVOPS-1052
Open

DEVOPS-1052: Add PR comment warning on force push to prompt Copilot re-review#180
RomFloreani wants to merge 1 commit into
mainfrom
DEVOPS-1052

Conversation

@RomFloreani
Copy link
Copy Markdown
Contributor

@RomFloreani RomFloreani commented May 26, 2026

DEVOPS-1052 - automatically trigger review by Copilot on force push

Repo where the tag was test: https://github.com/MiraGeoscience/JIRA/pulls

Copilot AI review requested due to automatic review settings May 26, 2026 19:25
@github-actions github-actions Bot changed the title Add PR comment warning on force push to prompt Copilot re-review DEVOPS-1052: Add PR comment warning on force push to prompt Copilot re-review May 26, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new job to the reusable Jira PR workflow that detects force pushes on synchronize events and posts a PR comment warning that prior Copilot reviews may be stale. Also tweaks the Jira summary and issue-check job conditions to handle bot vs user senders on synchronize and ready_for_review events.

Changes:

  • Add notify_force_push job that compares before/after SHAs via gh api and posts a warning comment on detected force pushes.
  • Extend add_jira_summary condition to also run on ready_for_review triggered by a user.
  • Restrict check_jira_issue so it does not run for bot-triggered ready_for_review events.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RomFloreani RomFloreani requested a review from sebhmg May 27, 2026 13:01
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