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

Full test coverage #14

Merged
merged 1 commit into from
Aug 17, 2024
Merged

Full test coverage #14

merged 1 commit into from
Aug 17, 2024

Conversation

chuckwondo
Copy link
Contributor

Increase test coverage to 100%, plus a bit of further tidying:

  • Use python -m pip instead of pip directly in Makefile to ensure correct installation of pip is used
  • Upgrade pip in CI to eliminate "newer pip available" message
  • Uniquely name test case that was a duplicate name (likely an earlier copy-paste error)

@chuckwondo
Copy link
Contributor Author

@francium, before I submit a PR for #9, I thought I would do a bit of tidying of test coverage to keep my PRs a bit more granular for you.

@francium francium merged commit 87a13c6 into rustedpy:master Aug 17, 2024
5 checks passed
@francium
Copy link
Member

👍

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.

2 participants