-
Notifications
You must be signed in to change notification settings - Fork 56
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
Make code compression optional, default to —compress=true. Fixes gh-776 #801
Conversation
compress: false, | ||
slim: true, | ||
}).then(bundle => { | ||
extract(bundle, (error, entries) => { |
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.
Looks like this is causing an linter error.
The code looks good to me (after the linter error is fixed). 👍 I can do a smoke test tonight but I'm fairly confident this is good to go. |
Strange... those lines don't match the code I'm looking at locally... |
Smoke test worked as it should. Also opened up a quick PR to this branch to fix that |
@rwaldron No problem. |
Yes, it will completely bypass the compression. |
d4f54e3
to
20c51f7
Compare
…elgh-776 - `t2 run <index.js> --compress=false` will skip the code compression step Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Thanks @rwaldron. |
Thanks for the review, this will be in the release today |
t2 run <index.js> --compress=false
will skip the code compression step@huseyinkozan sorry for the delay on this
@HipsterBrown @carlbaron review?
Smoke Test
mkdir t2-776 && cd t2-776 && t2 init
t2 run index.js --compress=false
t2 root
cd /tmp/remote-scripts/ && cat index.js
. The content should be the same as it appears in your local project directory.Signed-off-by: Rick Waldron waldron.rick@gmail.com