-
Notifications
You must be signed in to change notification settings - Fork 416
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
Ensure consistent artifact sha
(remove bestzip
and bring back archiver
)
#1018
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.
LGTM
f35818d
to
5668fcf
Compare
5668fcf
to
abf3f82
Compare
We tried to add e2e tests but we failed for now. |
9b59391
to
9308429
Compare
* | ||
* @see https://github.com/serverless/serverless/blob/63d54e1537e10ae63c171892edd886f6b81e83f6/lib/plugins/package/lib/zipService.js#L65 | ||
*/ | ||
function serverlessZip(args) { |
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.
It seems the code coverage dropped because there are no unit tests for this new function.
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.
It's hard to test unit it, in fact, we would like to provide e2e tests which will more easy to cover that code.
9308429
to
f9c99a5
Compare
f9c99a5
to
df24200
Compare
Replace #911
Closes #881