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

Fix GitHub pages #191

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix GitHub pages #191

wants to merge 2 commits into from

Conversation

Zen0x7
Copy link

@Zen0x7 Zen0x7 commented Jul 31, 2024

image

As peaceiris/actions-gh-pages README describes:

permissions:
    contents: write

is required. In other hand, without that, the step will fail.

Also a jump to the version 4 is recommended.

Basically because i did a test on the action without this change, just doesn't work:

image

Anyways, thanks so much @TheLartians.

Zen0x7 and others added 2 commits July 31, 2024 03:51
As `peaceiris/actions-gh-pages` README describes, the `permissions / contents / write` definition is required. In other hand, without that, the step will fail. Also a jump to the version 4 is recommended. Basically because i did a test and previous version, as-is doesn't work.
@TheLartians
Copy link
Owner

Thanks a lot for the fix @Zen0x7 and sorry for the late review! The changes look good, however it seems that GitHub Actions are not starting running in this PR for some reason, even after merging with main. Could it be that you need to enable Actions on your fork?

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