-
Notifications
You must be signed in to change notification settings - Fork 249
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
Compress files into a single coverage.zip and update upload logic accordingly #1528
Conversation
Diff Coverage: The code coverage on the diff in this pull request is 89.1%. Total Coverage: This PR will increase coverage by 1.32%. File Coverage Changes
🛟 Help
|
…cluded in the zip 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.
Nice simplification!
Instead of uploading 4 separate files for coverage uploads, this uploads a single zip file containing the 4 files.
This cannot be merged until a server side component which knows how to read the single zip file is merged.