ci(codecov): Fix uploading of python coverage #2360
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## edge #2360 +/- ##
===========================================
+ Coverage 30.02% 49.61% +19.59%
===========================================
Files 508 580 +72
Lines 8127 15220 +7093
===========================================
+ Hits 2440 7552 +5112
- Misses 5687 7668 +1981
Continue to review full report at Codecov.
|
sfoster1
approved these changes
Sep 25, 2018
b-cooper
added a commit
that referenced
this pull request
Sep 26, 2018
* fix(app): Open external links in browser instead of app window (#2327) * feat(api): Remove deck calibration from reset options (#2330) Remove deck calibration from factory reset options, as deck calibration reset should not be done outside of the context of recalibration * docs(contributing): Ask Windows users to set core.autocrlf=input (#2332) Setting core.autocrlf to something that inserts carriage returns on windows means that API pushes to robots from windows will have carriage returns in the scripts in api/opentrons/resources, which prevents the robot from booting. Closes #2305 * fix(api): Update definitions for tuberacks (#2317) * fix(api): Update definitions for 15ml tuberack, 50ml tuberack, and 15/50ml tuberack to match actual Closes #2290 * Modify definitions based on practical test * feat(app): Enable autoupdate on Linux by switching to AppImage builds (#2329) Closes #2303 * chore(chore): add exception in gitattributes for our hex files (#2339) * fix(protocol-designer): fix whitescreen on deleting blowout labware (#2341) * fix unticketed bug: setting blowout to a trash-box in a step and deleting the trash box caused whitescreen *make labware dropdown blank when nonexistent labware selected * fix(api): Update the aluminum block definitions to match drawings (#2342) * fix(api): Update the aluminum block definitions to match drawings and experiments * Rename definition in shared-data to match rename in default_containers.json and fix z height * Closes #2292 * chore(release): 3.4.0 (#2338) * fix(protocol-designer): fix tiprack diagram only displaying right (#2340) Fix a small bug where the tiprack diagram in the new file modal would duplicate the diagram of the selected right pipette tiprack instead of showing both selections. * feat(api): add module firmware update endpoint (#2173) Closes #1654 Adds firmware update api methods and update server functions (for modules with old as well as new bootloader) Adds a udev rule for new bootloader Adds tests for modules and update_server endpoint * feat(protocol-designer): add "app build date" field to PD saved files (#2350) * chore(api): Cleanup unused api root level files (#2336) There were some leftovers from days of old: we don't use pip or anaconda or PyInstaller anymore. * refactor(api): invert control of system and server (#2318) Closes #2185 Separates out the previously entangled server and system functions so that opentrons.main handles system configuration while server.main only configures and runs the server. Entrypoint is switched from opentrons.server.main to opentrons.main * feat(api): Add hardware_control submodule (#2349) The hardware_control submodule is the controller of the robot's hardware. Its API can move the robot at a level of abstraction that portrays the robot as a whole, not including the labware. In this PR, the module is a stub, and not used by default Closes #2232 * fixup: More verbose names for file and thread locks * fixup: use functools.wraps in log_call decorator * ci(codecov): Fix uploading of python coverage (#2360) * fix(protocol-designer): tweak analytics copy for accuracy (#2366) Update the copy in the analytics disclaimer to accurately portray our data collection in beta.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
overview
A while back our Python code coverage stopped getting uploaded. I had figured this was a problem with the codecov configuration, but it turns out it was just that a
make cleanwas wiping out the API's coverage report before codecov could find and upload it.This PR calls the coverage task directly after
make testto avoid any unintended report deletion and also switches from codecov's bash reporter to one fromnpmto avoid coverage (or build) breakages due to unexpected changes in Codecov's unversioned (for us) curl'd bash uploaderchangelog
review requests
Please check the coverage report for this branch - https://codecov.io/gh/Opentrons/opentrons/branch/build_fix-coverage