Skip to content

fix: gh-pages.yml permissions#5

Merged
tcharding merged 1 commit intorust-bitcoin:masterfrom
realeinherjar:patch-1
Oct 27, 2023
Merged

fix: gh-pages.yml permissions#5
tcharding merged 1 commit intorust-bitcoin:masterfrom
realeinherjar:patch-1

Conversation

@realeinherjar
Copy link
Copy Markdown
Contributor

We had the error on gh-pages actions workflow.
This might fix this.

Source: https://github.com/marketplace/actions/github-pages-action#%EF%B8%8F-first-deployment-with-github_token

If the action fails to push the commit or tag with the following error:

/usr/bin/git push origin gh-pages
remote: Write access to repository not granted.
fatal: unable to access 'https://github.com/username/repository.git/': The requested URL returned error: 403
Error: Action failed with "The process '/usr/bin/git' failed with exit code 128"

Please add the write permission to the permissions.contents in a workflow/job.

permissions:
  contents: write

We had the error on `gh-pages` actions workflow.
This might fix this.

Source: https://github.com/marketplace/actions/github-pages-action#%EF%B8%8F-first-deployment-with-github_token

> If the action fails to push the commit or tag with the following error:

> ```shell
> /usr/bin/git push origin gh-pages
> remote: Write access to repository not granted.
> fatal: unable to access 'https://github.com/username/repository.git/': The requested URL returned error: 403
> Error: Action failed with "The process '/usr/bin/git' failed with exit code 128"
> ```

> Please add the `write` permission to the `permissions.contents` in a workflow/job.

> ```yml
> permissions:
>   contents: write
> ```
Copy link
Copy Markdown
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

ACK 70f5fad

@tcharding tcharding merged commit 43a9c17 into rust-bitcoin:master Oct 27, 2023
@realeinherjar realeinherjar deleted the patch-1 branch October 27, 2023 10: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