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

chore: unpin pytest in CI #546

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

aaraney
Copy link
Member

@aaraney aaraney commented Mar 13, 2024

pytest==8.1.1 was failing to discover tests correctly because of an erroneous __init__.py file in the source tree (#545). Remove the pytest version pin and add the new consider_namespace_packages = true pytest option.

related to #541 and #542

#545 should be merged before this

@aaraney aaraney added the CI Issues related to the continuous integration testing. label Mar 13, 2024
@christophertubbs christophertubbs merged commit 549aa46 into NOAA-OWP:master Mar 13, 2024
8 checks passed
@aaraney
Copy link
Member Author

aaraney commented Mar 13, 2024

woot woot! Thanks, @christophertubbs!

@aaraney aaraney deleted the unpin-pytest branch March 13, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Issues related to the continuous integration testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants