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

Remove every file when cleaning up PR preview #17

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

EndBug
Copy link
Contributor

@EndBug EndBug commented Sep 2, 2022

Closes #15

Since JamesIves/github-pages-deploy-action does not currently provide a way to disable checks for .nojekyll and CNAME (see JamesIves/github-pages-deploy-action#1210), I've opted for running my add-and-commit action on temp worktree.
In the future, I may add an input that allows add-and-commit to create the worktree by itself, but for now, I'm doing it in two separate steps.

I've already tested the changes

Let me know what you think 😄

@rossjrw rossjrw added the bug Something isn't working label Sep 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup does not remove .nojekyll file
2 participants