Skip to content

Try to fix the PR CI job#9

Merged
xMAC94x merged 2 commits intoveloren:mainfrom
walpox:walpox/fix-ci
Apr 2, 2025
Merged

Try to fix the PR CI job#9
xMAC94x merged 2 commits intoveloren:mainfrom
walpox:walpox/fix-ci

Conversation

@walpox
Copy link
Contributor

@walpox walpox commented Apr 2, 2025

Changelog

The format is based on Keep a Changelog.

Fixed

  • Job permissions to bare minimum required.
  • Update GitHub CLI command with correct subcommand.
  • Change event trigger to pull_request_target to run in a trusted context.

@walpox walpox marked this pull request as draft April 2, 2025 14:53
The pull_request_target event runs on a trusted context, so hopefully
it will resolve the auth error.
@walpox walpox marked this pull request as ready for review April 2, 2025 15:00
@xMAC94x xMAC94x merged commit 6f687aa into veloren:main Apr 2, 2025
@walpox walpox deleted the walpox/fix-ci branch April 2, 2025 16:34
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