-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
feat(publish): clear built files before building package and packing #12516
feat(publish): clear built files before building package and packing #12516
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments--looking good!
Also my comments on _verbose can be ignored if there's a reason--just didn't know why to prefix with an underscore?
I'll close this is this has potential for problems and comment what I think is more straight forward solution in issue |
Argh, my bad, solution I thought of is not really possible (just output built files to |
Any updates on this @pieh? |
No updates on this yet, let me sync this branch and address Dustin's comments. And should be good to go I think |
df88676
to
10b0571
Compare
All right, updated the PR and is ready for review |
…atsbyjs#12516) * feat(publish): clear built files before building package and packing * Add some feedback message if dev doesn't confirm file deletion
This adds extra step before publishing to clear any previously built files.
Initially I added prompt to confirm file deletion just in case script need to be adjusted and problems can be caught
Example output:
https://gist.github.com/pieh/1ebb938eb300aaac882491bd94a3506f
Btw - this also catch some weirdness in our package build scripts - like
graphql-skip-limit
packs bothsrc
anddist
, or testfixtures being packed ingatsby-codemods
Closes #12491