Skip to content

Chore: [AEA-0000] - add workflow to sync copilot#2538

Merged
anthony-nhs merged 4 commits intomainfrom
sync_copilot
Apr 1, 2026
Merged

Chore: [AEA-0000] - add workflow to sync copilot#2538
anthony-nhs merged 4 commits intomainfrom
sync_copilot

Conversation

@anthony-nhs
Copy link
Copy Markdown
Collaborator

Summary

  • Routine Change

Details

  • add workflow to sync copilot

Copilot AI review requested due to automatic review settings March 30, 2026 16:42
@github-actions
Copy link
Copy Markdown
Contributor

This PR is linked to a ticket in an NHS Digital JIRA Project. Here's a handy link to the ticket:

AEA-0000

Copy link
Copy Markdown

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 scheduled/manual GitHub Actions workflow to sync shared Copilot instructions into this repository.

Changes:

  • Introduces .github/workflows/sync_copilot.yml to run on workflow_dispatch and a weekly cron schedule.
  • Runs the NHSDigital/eps-copilot-instructions action pinned to a commit SHA, passing refs and Create PR app credentials.

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

runs-on: ubuntu-22.04
environment: create_pull_request
permissions:
contents: read
Copy link

Copilot AI Mar 30, 2026

Choose a reason for hiding this comment

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

This job appears intended to open a PR (uses create_pull_request environment and CREATE_PULL_REQUEST_* secrets) but it only grants contents: read. In this repo, PR-creating automation (e.g. .github/workflows/update_dev_container_version.yml) also grants pull-requests: write (and, depending on how the action pushes commits, may require contents: write). Consider aligning the permissions here so the action can create/update a PR reliably.

Suggested change
contents: read
contents: write
pull-requests: write

Copilot uses AI. Check for mistakes.
tstephen-nhs
tstephen-nhs previously approved these changes Mar 31, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

@anthony-nhs anthony-nhs merged commit 96d134b into main Apr 1, 2026
33 of 34 checks passed
@anthony-nhs anthony-nhs deleted the sync_copilot branch April 1, 2026 13:54
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.

3 participants