Skip to content

Remove every file when cleaning up PR preview#17

Open
EndBug wants to merge 4 commits intorossjrw:mainfrom
EndBug:cleanup
Open

Remove every file when cleaning up PR preview#17
EndBug wants to merge 4 commits intorossjrw:mainfrom
EndBug:cleanup

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