Skip to content

Conversation

@bjlittle
Copy link
Member

This pull-request:

  • updates the optional developer dependencies to include pre-commit
  • updates the optional test dependencies to include nox, codecov, pytest-cov
  • enables codecov test coverage reporting for the latest supported distribution of python i.e., py312
  • rationalises the PyPI requirements

@codecov-commenter
Copy link

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 ☂️

@bjlittle
Copy link
Member Author

ping @tkknight 😉

@bjlittle bjlittle requested a review from pp-mo April 24, 2024 16:20
@tkknight tkknight self-requested a review April 25, 2024 13:48
@bjlittle bjlittle removed the request for review from pp-mo April 25, 2024 13:49
@bjlittle bjlittle assigned tkknight and unassigned pp-mo Apr 25, 2024
@tkknight tkknight merged commit c43905d into SciTools:main Apr 25, 2024
@bjlittle
Copy link
Member Author

@tkknight Awesome! Thanks 😄

Copy link
Member

@pp-mo pp-mo left a comment

Choose a reason for hiding this comment

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

After some thought, I do think I agree with all this !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants