Skip to content

Update GitHub Actions to latest stable versions and add skills#8062

Merged
hemarina merged 6 commits into
mainfrom
copilot/update-yml-files-github-action
May 6, 2026
Merged

Update GitHub Actions to latest stable versions and add skills#8062
hemarina merged 6 commits into
mainfrom
copilot/update-yml-files-github-action

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

Bump all GitHub Actions in workflow files to their latest stable major versions.

  • actions/checkout v4 → v6
  • actions/setup-node v4 → v6
  • actions/setup-python v4/v5 → v6
  • actions/setup-go v5 → v6
  • actions/github-script v5/v7 → v9
  • golangci/golangci-lint-action v8 → v9

Actions already at latest (upload-artifact@v4, stale@v10, devcontainers/action@v1, setup-terraform@v3) were left unchanged.

- actions/checkout@v4 → v6
- actions/setup-node@v4 → v6
- actions/setup-python@v4,v5 → v6
- actions/setup-go@v5 → v6
- actions/github-script@v5 → v7
- golangci/golangci-lint-action@v8 → v9

Agent-Logs-Url: https://github.com/Azure/azure-dev/sessions/c7e2c320-a9b0-4304-88e8-7b19ac8c025b

Co-authored-by: hemarina <104857065+hemarina@users.noreply.github.com>
Copilot AI changed the title [WIP] Update YML files to latest stable version of GitHub Action Update GitHub Actions to latest stable versions May 5, 2026
Copilot AI requested a review from hemarina May 5, 2026 23:01
Comment thread .github/workflows/vscode-ci.yml Outdated
Copilot AI requested a review from hemarina May 5, 2026 23:08
@hemarina hemarina marked this pull request as ready for review May 5, 2026 23:10
Copilot AI review requested due to automatic review settings May 5, 2026 23:10
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

This PR updates the repository’s GitHub Actions workflow dependencies by bumping referenced actions to newer major versions across CI, linting, governance, and release workflows.

Changes:

  • Bumped actions/checkout, actions/setup-node, actions/setup-go, and actions/setup-python major versions in multiple workflows.
  • Updated actions/github-script usages to v9 across workflows that rely on JavaScript helper scripts.
  • Updated golangci/golangci-lint-action to v9 in the Go lint workflow.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/vscode-ci.yml Bumps checkout/setup-node and github-script used by VS Code extension CI.
.github/workflows/validate-go-version.yml Bumps checkout version for the Go version consistency check.
.github/workflows/schema-ci.yml Bumps checkout/setup-node and setup-python for schema validation CI.
.github/workflows/pr-governance.yml Bumps checkout and github-script for PR governance automation.
.github/workflows/lint-go.yml Bumps checkout and golangci-lint action major versions.
.github/workflows/lint-bicep.yml Bumps checkout version for Bicep lint workflow.
.github/workflows/ext-registry-ci.yml Bumps checkout version for extension registry CI workflow.
.github/workflows/eval-unit.yml Bumps checkout/setup-go/setup-node/setup-python for eval unit test workflow.
.github/workflows/devops-ext-ci.yml Bumps checkout/setup-node for Azure DevOps extension CI.
.github/workflows/devcontainer-feature-test.yml Bumps checkout for devcontainer feature test workflows.
.github/workflows/devcontainer-feature-release.yml Bumps checkout for devcontainer feature release workflow.
.github/workflows/cspell-misc.yml Bumps checkout/setup-node for misc spellcheck workflow.
.github/workflows/cspell-ext.yml Bumps checkout/setup-node for extension spellcheck workflow.
.github/workflows/copilot-setup-steps.yml Bumps checkout/setup-node for Copilot coding agent environment setup.
.github/workflows/cli-ci.yml Bumps checkout/setup-node for CLI CI and misc checks.
.github/workflows/approval-ext-azure-ai-agents.yml Bumps checkout and updates github-script reference for approval gate workflow.

Comment thread .github/workflows/approval-ext-azure-ai-agents.yml
Copy link
Copy Markdown
Member

@jongio jongio left a comment

Choose a reason for hiding this comment

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

Version bumps all check out - confirmed v6 exists for checkout/setup-node/setup-python/setup-go, and v9 exists for github-script and golangci-lint-action.

Checked the scripts in .github/scripts/ for compatibility with github-script@v9 (which drops require('@actions/github') support). All scripts use the standard injected {github, context, core} pattern, so no breakage expected there.

One concern flagged inline: the SHA pin removal in the approval gate workflow.

Comment thread .github/workflows/approval-ext-azure-ai-agents.yml
Copy link
Copy Markdown
Member

@vhvb1989 vhvb1989 left a comment

Choose a reason for hiding this comment

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

It would be nice if you can create one skill in the repo to check if any of these needs update and doing and update

Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@wbreza wbreza left a comment

Choose a reason for hiding this comment

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

Clean dependency update — looks good. A couple of minor observations to supplement existing reviews:

  1. Pinning style inconsistency (low): approval-ext-azure-ai-agents.yml uses SHA pinning for github-script (@3a2844b7…) while pr-governance.yml and vscode-ci.yml use tag pinning (@v9). Pre-existing, but could be normalized in a follow-up.

  2. Completeness (info): 16/23 workflows updated. The 5 lint-ext-*.yml files are covered transitively via lint-go.yml. Worth confirming event.yml doesn't need a checkout bump as well.

jongio's compatibility verification for github-script@v9 checks out — all scripts use the standard injected {github, context, core} pattern.

Copy link
Copy Markdown
Member

@jongio jongio left a comment

Choose a reason for hiding this comment

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

SHA pin restored on the approval gate workflow. All version bumps verified, scripts compatible with github-script v9.

@hemarina
Copy link
Copy Markdown
Contributor

hemarina commented May 6, 2026

@copilot Check if event.yml need a checkout bump as well.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 6, 2026

@copilot Check if event.yml need a checkout bump as well.

event.yml does not use actions/checkout — it only uses azure/azure-sdk-actions@main. No bump needed there.

@hemarina hemarina enabled auto-merge (squash) May 6, 2026 21:51
@hemarina hemarina changed the title Update GitHub Actions to latest stable versions Update GitHub Actions to latest stable versions and add skills May 6, 2026
Copy link
Copy Markdown
Member

@jongio jongio left a comment

Choose a reason for hiding this comment

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

The latest commit reverts the SHA pin that was just added to address my previous feedback. This contradicts the tool-version-upgrade SKILL.md updated in the same push, which says 'Never demote a SHA pin to a bare tag' and cites this exact file as a supply-chain hardening example.

Comment thread .github/workflows/approval-ext-azure-ai-agents.yml
Copy link
Copy Markdown
Member

@trangevi trangevi left a comment

Choose a reason for hiding this comment

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

Approved for agents

Copy link
Copy Markdown
Contributor

@wbreza wbreza left a comment

Choose a reason for hiding this comment

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

Re-review after commit 5 ("add skills")

The original workflow version bumps (commits 1–4) remain clean — my prior APPROVE still stands for those changes.

New observations on commit 5:

  1. Scope expansion (medium): Commit 5 adds a substantial rewrite of .github/skills/tool-version-upgrade/SKILL.md (+280/−14 lines), documenting GitHub Actions as a new supported tool category. This is a significant addition not reflected in the PR description — consider updating the body to cover this.

  2. Separate concern (low): The SKILL.md documentation rewrite is a distinct change from the workflow bumps. For cleaner review history, this could have been a separate PR, but it's not blocking.

  3. Approval coverage (info): All three prior approvals (vhvb1989, wbreza, jongio) were submitted before commit 5. Worth confirming reviewers are aware of the expanded scope.

The SKILL.md content itself looks well-structured — covers SHA pin handling, YAML validation, inventory scanning, and PR templates for GitHub Actions upgrades.

Copy link
Copy Markdown
Contributor

@wbreza wbreza left a comment

Choose a reason for hiding this comment

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

Re-approving after reviewing commit 5 ("add skills"). The SKILL.md expansion is well-structured and the workflow bumps remain clean. LGTM.

@hemarina
Copy link
Copy Markdown
Contributor

hemarina commented May 6, 2026

/check-enforcer override

@hemarina hemarina merged commit 2983b00 into main May 6, 2026
34 of 38 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.

update yml files to latest stable version of GitHub Action

7 participants