Skip to content

Use PAT for publishing docs#96

Merged
Gudahtt merged 2 commits intomainfrom
use-new-docs-publishing-setup
Apr 11, 2023
Merged

Use PAT for publishing docs#96
Gudahtt merged 2 commits intomainfrom
use-new-docs-publishing-setup

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Apr 4, 2023

The publish-docs workflow has been updated to use a personal access token for publishing docs instead of using the GITHUB_TOKEN. This allows for more control over doc publishing permissions.

Relates to MetaMask/metamask-module-template#184

@Gudahtt

This comment was marked as resolved.

@Gudahtt
Copy link
Copy Markdown
Member Author

Gudahtt commented Apr 4, 2023

This requires some changes to the repository settings, but I will wait until just prior to merge before making those changes to avoid disrupting other PRs.

Typically I would test action changes in a fork, but in this case it's challenging because we're relying upon a feature not available without a paid plan (the ability to let metamaskbot bypass the review requirement), so I can't test it.

Base automatically changed from update-github-actions to main April 5, 2023 12:37
The `publish-docs` workflow has been updated to use a personal access token
for publishing docs instead of using the `GITHUB_TOKEN`. This allows for more
control over doc publishing permissions.

Relates to MetaMask/metamask-module-template#184
@Gudahtt Gudahtt force-pushed the use-new-docs-publishing-setup branch from 2a903aa to 2198d49 Compare April 5, 2023 12:38
@Gudahtt Gudahtt marked this pull request as ready for review April 5, 2023 12:38
@Gudahtt Gudahtt requested a review from a team as a code owner April 5, 2023 12:38
@Gudahtt
Copy link
Copy Markdown
Member Author

Gudahtt commented Apr 11, 2023

The required settings changes (as documented here) have been made

@Gudahtt Gudahtt merged commit 9d61c92 into main Apr 11, 2023
@Gudahtt Gudahtt deleted the use-new-docs-publishing-setup branch April 11, 2023 13: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