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.

Sign up for GitHub

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

Jump to bottom

Remove Codecov as a CI step on master/production branches #3708

Closed
kokrui opened this issue Apr 3, 2024 · 0 comments · Fixed by #3743
Closed

Remove Codecov as a CI step on master/production branches #3708

kokrui opened this issue Apr 3, 2024 · 0 comments · Fixed by #3743
Labels
devops reliability, supportability, infrastructure good first issue

Comments

@kokrui
Copy link
Member

kokrui commented Apr 3, 2024 •
edited
Loading

As mentioned in #3688, I think we should remove Codecov as a CI step on the master and production branches.

I think we should keep it on pull-requests so we can still see how coverage changes each time a PR is made / while reviewing PRs, but once a commit has landed in master / prod it's no longer useful information.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
devops reliability, supportability, infrastructure good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Codecov on master and production branches itstrueitstrueitsrealitsreal/nusmods
1 participant
@kokrui