-
Notifications
You must be signed in to change notification settings - Fork 344
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(core): partial scripts TS conversion #3353
chore(core): partial scripts TS conversion #3353
Conversation
archiver.directory(file, file); | ||
} else { | ||
archiver.file(file, { name: file }); |
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.
These lines used to omit the second argument to archive.directory
and archive.file
. I read the source code for the archiver
package and determined that passing the same path in the second argument would yield the equivalent behavior.
258dfc6
to
d836ea4
Compare
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.
validated these run normally on my machine, seems good! thanks!
Explanation of Bugfix/Feature/Modification:
This is a partial conversion attempt for the
scripts
dir in the repo root. I wanted to stop before the diff got too out of control.This accomplishes the following:
ts-node
dependency.scripts/lib
dir based on some commonalities observed between all of the scriptscompressSrc.ts
andcompressDist.ts
Once this is merged, I'll work on the remaining
scripts
dir, and then move on to the application code.I ran all of the scripts that I converted, they seemed to do what they had before conversion/refactoring.