ci/setup #2
ci/setup #2
Conversation
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Hallfred
left a comment
Hallfred
left a comment
There was a problem hiding this comment.
Seems to work, I commented some observations
There was a problem hiding this comment.
This deviates somewhat from the setup used in vortex-auv, in that it here runs on pull_request, and the job name being coverage_build as opposed to call_reusable_workflow
There was a problem hiding this comment.
Yeah I checked on some other repositories and they have their code coverage to run on pull requests also, so I just copied that, but should update vortex-auv too. And should change name true true
There was a problem hiding this comment.
Unit test paths are not defined
There was a problem hiding this comment.
It seemed to work fine without needing to specify it, I think its only if the filestructure is a mess (like vortex-auv)
|
🎉 This PR is included in version 1.4.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Add CI/CD pipeline for Industrial CI and Code coverage. Also add status badges.