Skip to content

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

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

Chore: [AEA-0000] - Sync copilot#80
anthony-nhs merged 6 commits intomainfrom
sync_copilot

Conversation

@anthony-nhs
Copy link
Copy Markdown
Contributor

Summary

  • Routine Change

Details

  • workflow to sync copilot

Copilot AI review requested due to automatic review settings April 1, 2026 14:04
@anthony-nhs anthony-nhs merged commit 8784807 into main Apr 1, 2026
4 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

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

Updates the scheduled GitHub Actions workflow responsible for syncing shared Copilot instructions.

Changes:

  • Bumps the pinned NHSDigital/eps-copilot-instructions action SHA.
  • Removes the workflow-level permissions: {} block.

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

@@ -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.
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