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

Remove deprecated codecov dependency from CI #1494

Merged
merged 10 commits into from
Sep 18, 2021
Merged

Conversation

weiji14
Copy link
Member

@weiji14 weiji14 commented Sep 12, 2021

Description of proposed changes

The codecov python package is deprecated in favour of a bash script, so it should be removed from PyGMT's CI dependency lists. See codecov/codecov-python#325 and https://about.codecov.io/blog/codecov-uploader-deprecation-plan/ for more information.

Note that PyGMT's code coverage reports are currently generated using pytest-cov (which produces a coverge.xml file), with the reported uploaded to https://app.codecov.io/gh/GenericMappingTools/pygmt/ using codecov-action. So there hasn't been any real use of installing the codecov PyPI package on local development environments.

Closes #1482

Reminders

  • Run make format and make check to make sure the code follows the style guide.
  • Add tests for new features or tests that would have caught the bug that you're fixing.
  • Add new public functions/methods/classes to doc/api/index.rst.
  • Write detailed docstrings for all functions/methods.
  • If adding new functionality, add an example to docstrings or tutorials.

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash commands are:

  • /format: automatically format and lint the code
  • /test-gmt-dev: run full tests on the latest GMT development version

@weiji14 weiji14 added the maintenance Boring but important stuff for the core devs label Sep 12, 2021
@weiji14 weiji14 self-assigned this Sep 12, 2021
@weiji14 weiji14 force-pushed the codecov-v2-uploader branch from b468d31 to dedf242 Compare September 12, 2021 10:31
@weiji14 weiji14 changed the title WIP: Fix code coverage uploads with codecov v2 Remove deprecated codecov dependency from CI Sep 17, 2021
@weiji14 weiji14 marked this pull request as ready for review September 17, 2021 09:52
Copy link
Member

@seisman seisman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@seisman seisman added this to the 0.5.0 milestone Sep 17, 2021
@weiji14 weiji14 added the final review call This PR requires final review and approval from a second reviewer label Sep 17, 2021
@weiji14 weiji14 merged commit 13f69b1 into main Sep 18, 2021
@weiji14 weiji14 deleted the codecov-v2-uploader branch September 18, 2021 10:15
@weiji14 weiji14 removed the final review call This PR requires final review and approval from a second reviewer label Sep 18, 2021
sixy6e pushed a commit to sixy6e/pygmt that referenced this pull request Dec 21, 2022
The codecov python package is deprecated in favour of a bash script,
so it should be removed from PyGMT's CI dependency lists. For more
info, see codecov/codecov-python#325 and
https://about.codecov.io/blog/codecov-uploader-deprecation-plan.

* Refresh codecov badge to be the official one
* Drop deprecated codecov dependency from conda install
* Add note to maintenance.md on how code coverage reports are handled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Boring but important stuff for the core devs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Codecov GitHub checks not running?
2 participants