Skip to content

Chore: [AEA-0000] - Sync copilot #80

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
anthony-nhs merged 6 commits into from
Apr 1, 2026
+1 −3
Merged

Chore: [AEA-0000] - Sync copilot #80

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4a2fbef
sync copilot
anthony-nhs Mar 30, 2026
cf6b089
Merge remote-tracking branch 'origin/main' into sync_copilot
anthony-nhs Apr 1, 2026
a883c1f
add pre-commit to check signing
anthony-nhs Apr 1, 2026
9f9b7b3
Merge branch 'main' into sync_copilot
anthony-nhs Apr 1, 2026
1a86340
Merge remote-tracking branch 'origin/main' into sync_copilot
anthony-nhs Apr 1, 2026
cc7a402
fix
anthony-nhs Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions .github/workflows/sync_copilot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ on:
schedule:
- cron: '0 6 * * 1'

Copy link

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

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

This workflow no longer defines workflow-level permissions: {}. In this repo, other workflows set permissions: {} at the top level (e.g. .github/workflows/ci.yml:5, pull_request.yml:6, release.yml:6) and then grant only the required permissions per-job. Keeping the workflow-level permissions explicitly locked down helps avoid accidentally inheriting broader default GITHUB_TOKEN permissions if additional jobs/steps are added later; consider restoring the top-level permissions: {} (or an explicitly minimal set) and relying on the existing job-level permissions overrides.

Suggested change
permissions: {}

Copilot uses AI. Check for mistakes.
permissions: {}

jobs:
sync-copilot-instructions:
runs-on: ubuntu-22.04
Expand All @@ -16,7 +14,7 @@ jobs:

steps:
- name: Sync shared instructions
uses: NHSDigital/eps-copilot-instructions@304ab2f4b7cdc15a1d7c0a0fae5290fad41b2451
uses: NHSDigital/eps-copilot-instructions@a7849a16aabd5c1edef13e29467a480fa08555f8
with:
copilot_instructions_ref: main
calling_repo_base_branch: main
Expand Down
Loading