Skip to content

Chore: [AEA-0000] - use action to sync copilot#1048

Merged
anthony-nhs merged 3 commits intomainfrom
fix_sync
Mar 30, 2026
Merged

Chore: [AEA-0000] - use action to sync copilot#1048
anthony-nhs merged 3 commits intomainfrom
fix_sync

Conversation

@anthony-nhs
Copy link
Copy Markdown
Contributor

Summary

  • Routine Change

Details

  • use action to sync copilot

Copilot AI review requested due to automatic review settings March 30, 2026 15:46
@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

Updates the repository’s “Sync Copilot Instructions” GitHub Actions workflow to use a dedicated action for syncing shared Copilot instructions (and opening a PR), rather than calling a reusable workflow.

Changes:

  • Replace reusable-workflow invocation with a single job that runs the sync action and can create PRs.
  • Update the scheduled trigger time and simplify workflow_dispatch (remove custom inputs).
  • Add job-level permissions/environment needed for PR creation.

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

with:
common_workflows_ref: ${{ inputs.common_workflows_ref }}
sync-copilot-instructions:
runs-on: ubuntu-latest
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.

runs-on: ubuntu-latest is the only workflow runner in this repo that tracks the moving ubuntu-latest label; the other workflows pin to ubuntu-22.04. Pinning the runner here as well (e.g., ubuntu-22.04) avoids unexpected breakages when GitHub updates the default image.

Suggested change
runs-on: ubuntu-latest
runs-on: ubuntu-22.04

Copilot uses AI. Check for mistakes.

steps:
- name: Sync shared instructions
uses: NHSDigital/eps-copilot-instructions@initial_instructions
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.

The action reference NHSDigital/eps-copilot-instructions@initial_instructions is a mutable ref (branch/tag), which is a supply-chain risk and can change behavior without this repo changing. Prefer pinning to an immutable commit SHA (or at least a protected, versioned tag) similar to other workflows in this repo.

Suggested change
uses: NHSDigital/eps-copilot-instructions@initial_instructions
uses: NHSDigital/eps-copilot-instructions@0123456789abcdef0123456789abcdef01234567

Copilot uses AI. Check for mistakes.
@anthony-nhs anthony-nhs temporarily deployed to create_pull_request March 30, 2026 16:19 — with GitHub Actions Inactive
@sonarqubecloud
Copy link
Copy Markdown

@anthony-nhs anthony-nhs merged commit 83655c6 into main Mar 30, 2026
16 of 17 checks passed
@anthony-nhs anthony-nhs deleted the fix_sync branch March 30, 2026 16:24
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