Bump actions/github-script from 8.0.0 to 9.0.0#1279
Merged
aaronpowell merged 1 commit intoMay 15, 2026
Conversation
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1279Or
iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1279" |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates GitHub Actions workflows in this repo to use actions/github-script v9.0.0 (from v8.0.0), aligning CI automation with the latest major release of the action.
Changes:
- Bump
actions/github-scripttov9.0.0inremove-stale-label.yml. - Update pinned
actions/github-scriptcommit SHA to the v9.0.0 release commit indogfood-comment.ymlandaspire-upgrade.lock.yml.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/remove-stale-label.yml | Bumps actions/github-script to v9.0.0 for stale-label removal automation. |
| .github/workflows/dogfood-comment.yml | Updates the pinned actions/github-script SHA to v9.0.0 for PR dogfood comment automation. |
| .github/workflows/aspire-upgrade.lock.yml | Updates the pinned actions/github-script SHA to v9.0.0 across multiple gh-aw steps. |
Copilot's findings
Comments suppressed due to low confidence (3)
.github/workflows/remove-stale-label.yml:34
- For
issue_commentevents on PRs, the payload typically doesn't includecontext.payload.pull_request.number; the PR number is available ascontext.payload.issue.number. Usingcontext.payload.pull_request.numberhere will likely beundefinedand causeremoveLabelto fail. Use the issue number from theissueobject for PR comments as well.
github.rest.issues.removeLabel({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: context.payload.pull_request.number,
name: 'stale'
.github/workflows/remove-stale-label.yml:35
- The
github.rest.issues.removeLabel(...)call returns a promise but isn't awaited. That can cause the step to finish without surfacing API failures reliably (e.g., permission errors / missing label). Await the call so failures fail the workflow deterministically.
This issue also appears on line 42 of the same file.
github.rest.issues.removeLabel({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: context.payload.pull_request.number,
name: 'stale'
});
.github/workflows/remove-stale-label.yml:47
- This
github.rest.issues.removeLabel(...)call isn't awaited, so API failures may not be surfaced and the step can behave inconsistently. Await the promise so the workflow fails deterministically on errors.
github.rest.issues.removeLabel({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: context.payload.issue.number,
name: 'stale'
});
- Files reviewed: 3/3 changed files
- Comments generated: 0
Member
|
@dependabot rebase |
Bumps [actions/github-script](https://github.com/actions/github-script) from 8.0.0 to 9.0.0. - [Release notes](https://github.com/actions/github-script/releases) - [Commits](actions/github-script@v8...v9) --- updated-dependencies: - dependency-name: actions/github-script dependency-version: 9.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
873ed49 to
a7709ce
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps actions/github-script from 8.0.0 to 9.0.0.
Release notes
Sourced from actions/github-script's releases.
Commits
3a2844bMerge pull request #700 from actions/salmanmkc/expose-getoctokit + prepare re...ca10bbdfix: use@octokit/core/types import for v7 compatibility86e48e2merge: incorporate main branch changesc108472chore: rebuild dist for v9 upgrade and getOctokit factoryafff112Merge pull request #712 from actions/salmanmkc/deployment-false + fix user-ag...ff8117eci: fix user-agent test to handle orchestration ID81c6b78ci: use deployment: false to suppress deployment noise from integration tests3953cafdocs: update README examples from@v8to@v9, add getOctokit docs and v9 brea...c17d55bci: add getOctokit integration test joba047196test: add getOctokit integration tests via callAsyncFunction