Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to actions/github-script@v7 #776

Merged
merged 2 commits into from
Nov 15, 2023

Conversation

dscho
Copy link
Member

@dscho dscho commented Nov 14, 2023

This imitates what I just did in get-azure-pipelines-artifact, and it only affects the release-tag workflow (i.e. it does not require a new tag and release).

Now that this Action uses node.js 20, let's also use it in the steps of
the CI builds which use `github-script`.

Signed-off-by: Johannes Schindelin <[email protected]>
It is no longer enough to call `github.users.listGpgKeysForUser()`, it
needs to specify the REST API mode (as opposed to GraphQL):
`github.rest.users.listGpgKeysForUser()`.

This was highly recommended before upgrading to
`actions/github-script@v7`, but now is required.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho requested a review from rimrul November 14, 2023 23:40
@dscho dscho self-assigned this Nov 14, 2023
@dscho dscho merged commit 44e6ddb into git-for-windows:main Nov 15, 2023
2 checks passed
@dscho dscho deleted the update-to-github-script@v7 branch November 15, 2023 07:17
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