Skip to content

Chore: [AEA-0000] - sync copilot#720

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

Chore: [AEA-0000] - sync copilot#720
anthony-nhs merged 2 commits intomainfrom
sync_copilot

Conversation

@anthony-nhs
Copy link
Copy Markdown
Contributor

Summary

  • Routine Change

Details

  • sync copilot

Copilot AI review requested due to automatic review settings March 30, 2026 16:56
@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 GitHub Copilot instructions into this repository (via a dedicated action).

Changes:

  • Introduces .github/workflows/sync_copilot.yml to run weekly (and on-demand) to sync shared Copilot instructions.
  • Configures the workflow to use the NHSDigital/eps-copilot-instructions action with app-based 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.

sync-copilot-instructions appears to be intended to create a PR (it is using CREATE_PULL_REQUEST_* credentials), but the job permissions only grant contents: read. In this repo, other PR-creating automation (e.g. .github/workflows/update_dev_container_version.yml:11-15) also requests pull-requests: write. Add pull-requests: write here (and any other minimal permissions required by the action) to avoid the workflow failing when it tries to open/update a PR.

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

Copilot uses AI. Check for mistakes.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

@anthony-nhs anthony-nhs merged commit e0bb242 into main Apr 1, 2026
12 checks passed
@anthony-nhs anthony-nhs deleted the sync_copilot branch April 1, 2026 13:48
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