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
chore(package.json): clear up intermidiate build artifact after build #2206
Conversation
I'm feeling bulid script might need to be simplified (and remove some of confusions if anyone read scripts).. but that's other story later. |
"build_perf": "webdriver-manager update && npm-run-all build_cjs build_global perf", | ||
"build_test": "shx rm -rf ./dist/ && npm-run-all build_cjs clean_spec build_spec test_mocha", | ||
"build_cover": "shx rm -rf ./dist/ && npm-run-all build_cjs build_spec cover", | ||
"build_docs": "npm-run-all build_global build_es6_for_docs build_cjs clean_spec build_spec tests2png decision_tree_widget && esdoc -c esdoc.json", | ||
"build_docs": "npm-run-all build_global build_es6_for_docs build_cjs clean_spec build_spec decision_tree_widget && esdoc -c esdoc.json && npm-run-all clean_dist_es6", |
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.
should tests2png have been kept? By removing it here, it's no longer used in any other script and I think it is indeed needed as part of the doc building process?
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.
OH NO 😱 my bad, I was removing it to test script without png generation tools setup.
077cc2e
to
3a59884
Compare
Updated PR corrects my error. |
Seems good, but I want @Blesh to confirm as well. |
@jayphelps I agree. |
LGTM |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description:
This PR makes force clear intermediate build artifact
es6
used for internal doc / umd package building only, to no longer included in actualRelated issue (if exists):
#2177