Skip to content

Chore: [AEA-6527] - fix permissions so can publish packages#127

Merged
tstephen-nhs merged 8 commits intomainfrom
aea-6257-fix-package-perms
Apr 7, 2026
Merged

Chore: [AEA-6527] - fix permissions so can publish packages#127
tstephen-nhs merged 8 commits intomainfrom
aea-6257-fix-package-perms

Conversation

@tstephen-nhs
Copy link
Copy Markdown
Contributor

@tstephen-nhs tstephen-nhs commented Apr 7, 2026

Summary

  • 🤖 Operational or Infrastructure Change

Details

  • fix permissions to publish packages

Copilot AI review requested due to automatic review settings April 7, 2026 10:12
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 reusable GitHub Actions workflows to satisfy actionlint syntax requirements and to support publishing to GitHub Packages from the release workflow.

Changes:

  • Replace YAML step aliases in steps: arrays with explicit step mappings to satisfy actionlint.
  • Add packages: write permission to the tag-release devcontainer job to support package publishing.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
.github/workflows/tag-release-devcontainer.yml Adds package write permission to enable publishing from the release job.
.github/workflows/quality-checks-devcontainer.yml Expands previously-aliased steps into explicit step definitions to satisfy actionlint.

anthony-nhs
anthony-nhs previously approved these changes Apr 7, 2026
@tstephen-nhs tstephen-nhs enabled auto-merge (squash) April 7, 2026 12:59
@tstephen-nhs tstephen-nhs changed the title Chore: [AEA-6527] - fix actionlint error by replacing alias step entries with explicit step mappings Chore: [AEA-6527] - fix permissions so can publich packages Apr 7, 2026
@tstephen-nhs tstephen-nhs changed the title Chore: [AEA-6527] - fix permissions so can publich packages Chore: [AEA-6527] - fix permissions so can publish packages Apr 7, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@tstephen-nhs tstephen-nhs merged commit c8f899f into main Apr 7, 2026
11 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.

3 participants