-
Notifications
You must be signed in to change notification settings - Fork 46
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
Cleanup does not remove .nojekyll file #15
Comments
Funnily enough, I can think of a reason why. This is behaviour specific to JamesIves/github-pages-deploy-action, which this action uses under the hood. JamesIves' action is intended for traditional single-site deployments, and intentionally does not remove CNAME and .nojekyll files during its clean process. You can see the reasoning here: JamesIves/github-pages-deploy-action#354 However, for a removed preview, these files are definitely not useful, so this action should take care of removing them as well. Therefore I'm going to consider this a bug. It'll be a while before I can get around to working on this again, so in the meantime you might have some luck with detecting the preview path yourself and adding an extra step to your removal workflow to delete those files. Or, if you feel comfortable with it, I'd gladly accept a PR. |
Closes rossjrw#15 Related future improvements: JamesIves/github-pages-deploy-action#1210 EndBug/add-and-commit#429
Hi, thanks for the awesome action!
I was just testing it on a repo and I found out that when the PR is closed the preview is correctly removed, but a
.nojekyll
file hangs around in the directory. Do you know why?Here are some links:
Thanks for the help ❤️
The text was updated successfully, but these errors were encountered: