Skip to content
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

[6.15.z] Remove codecov #1126

Merged

Conversation

Gauravtalreja1
Copy link
Collaborator

Description of changes

Codecov was removed from master as its no longer required in PR #1102
We're missing this changes in zstream branches and codecov checks fails on zstream PRs, so removing this from all supported branches
Example: #1124

This continues a trend we've done with other projects.
Codecov has some use in other types of projects, but really isn't a good
fit for Nailgun since it is functionally exercised by Robottelo.

Without getting too deep into philosophy, CodeCov is a misleading metric
for quality and minor changes are largely ignored in mature projects.

(cherry picked from commit ef2c9e7)
@Gauravtalreja1 Gauravtalreja1 added CherryPick PR needs CherryPick to previous branches 6.12.z 6.13.z 6.14.z labels Apr 9, 2024
@Gauravtalreja1 Gauravtalreja1 self-assigned this Apr 9, 2024
@Griffin-Sullivan Griffin-Sullivan merged commit 8254538 into SatelliteQE:6.15.z Apr 9, 2024
22 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the remove-codecov-zstream branch April 9, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.12.z 6.13.z 6.14.z CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants