🐛 Auto release github action failure due to missing permissions
#506
Conversation
This file contains 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
…te-release-versions ci: add release.yml to help automate release versions
…urther to cover edge cases
…te-release-versions ci: add release.yml to help automate release versions - edited file f…
…in a line which pointed at a different variable
…te-release-versions ci: add release.yml to help automate release versions - I had a typo …
…lease portion to prepend a v
…te-release-versions ci: add release.yml to help automate release versions - Fixing the re…
…o it looks like v1.1.0 instead of V1.1.0
…te-release-versions ci: add release.yml to help automate release versions. Lowercased V s…
…te-release-versions ci: changing structure of release.yml to be easier to understand
…te-release-versions ci: #minor changes made
…te-release-versions ci: #minor changes made to make rolling and stable both release branches
…te-release-versions ci: #minor changes to hopefully turn off pre-release
…ate-release-versions ci: #minor change - trying to make pre-release suffic set to nothing
…ate-release-versions ci: #minor change - removed older release action for a better up to d…
…ate-release-versions ci: #minor change
…ate-release-versions ci: #minor change to fixed the pre-release issue
…ate-release-versions ci: #minor change to fixed the pre-release issue again
…ate-release-versions ci: #minor change to fixed the pre-release issue again2
… what branch it targets
Co-authored-by: alamin655 <129589283+alamin655@users.noreply.github.com>
… issue with cargo.toml
neon-mmd
changed the title
neon-mmd
changed the title
github action failure due to missing permissions
neon-mmd
approved these changes
Feb 2, 2024
alamin655
approved these changes
Feb 3, 2024
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.
What does this PR do?
This PR provides a fix for the auto release GitHub action proposed in the PR #499 by replacing the GITHUB_TOKEN with the ADMIN_RIGHTS_TOKEN secret.
Why is this change important?
This change is essential as it provides a fix for the auto release GitHub action.
How to test this PR locally?
Place this file in the same location (.github/workflows/release.yml) then successfully merge a PR into the rolling branch. Once this is done you should see the action successfully runs and in a moment you will see the new tag and release. Please create a Personal Access Token or PAT and assign it these settings below. Not pictured is only setting the token to have control of the websurfx repo (or your fork).
Once you are done, please copy the token since you will not be able to see it again. Then you must add the PAT as a secret called ADMIN_RIGHTS_TOKEN. Please paste in the token you copied from above here. You may follow these steps https://docs.github.com/en/actions/security-guides/using-secrets-in-github-actions?tool=webui#creating-secrets-for-a-repository
Note: I used webui screenshots and notes to be more beginner-friendly, but you can do this in the CLI as well.
Author's checklist
N/A
Related issues
None