Skip to content

Chore: [AEA-0000] - workflow to update dev container#2908

Merged
anthony-nhs merged 4 commits intomainfrom
workflow_to_update_dev_container
Mar 27, 2026
Merged

Chore: [AEA-0000] - workflow to update dev container#2908
anthony-nhs merged 4 commits intomainfrom
workflow_to_update_dev_container

Conversation

@anthony-nhs
Copy link
Copy Markdown
Contributor

Summary

  • Routine Change

Details

  • workflow to update dev container

Copilot AI review requested due to automatic review settings March 25, 2026 09:47
@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
Contributor

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 that delegates to the shared eps-common-workflows reusable workflow to automatically update the devcontainer version on main.

Changes:

  • Introduces .github/workflows/update_dev_container_version.yml to run on a weekly cron and via workflow_dispatch
  • Calls the shared update-dev-container-version.yml reusable workflow with base_branch: main and automerge secrets

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


jobs:
update_devcontainer_version:
uses: NHSDigital/eps-common-workflows/.github/workflows/update-dev-container-version.yml@d2a4595b1bfa59f04c7cecb080a126321d42a333
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

This repository pins all other NHSDigital/eps-common-workflows reusable workflows to commit 5ac2707dd9cd60ad127275179495b9c890d74711, but this new workflow is pinned to d2a4595b.... If the newer ref isn’t required for update-dev-container-version.yml, consider aligning to the same commit as the other workflows (or bumping them together) to avoid drifting behavior and make updates easier to manage.

Suggested change
uses: NHSDigital/eps-common-workflows/.github/workflows/update-dev-container-version.yml@d2a4595b1bfa59f04c7cecb080a126321d42a333
uses: NHSDigital/eps-common-workflows/.github/workflows/update-dev-container-version.yml@5ac2707dd9cd60ad127275179495b9c890d74711

Copilot uses AI. Check for mistakes.
Comment on lines +12 to +14
contents: read
packages: read
pull-requests: write
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

permissions: block indentation is inconsistent with the rest of the workflows in this repo (they use two-space nesting under permissions:). Please align the indentation for readability and consistency.

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

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

@anthony-nhs anthony-nhs merged commit 93e8a87 into main Mar 27, 2026
63 of 66 checks passed
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