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

Update ioos-qc package #427

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update ioos-qc package #427

wants to merge 1 commit into from

Conversation

leilabbb
Copy link
Contributor

Updated to the new release: v2.2.0

Updated to the new release: v2.2.0
@leilabbb leilabbb added the QC label Feb 26, 2025
@leilabbb leilabbb requested a review from benjwadams February 26, 2025 16:30
@leilabbb leilabbb linked an issue Feb 26, 2025 that may be closed by this pull request
@benjwadams
Copy link
Contributor

Can you check the type hints in the unit tests?

@@ -26,8 +26,7 @@ thredds_crawler>=1.5.4
pyyaml>=5.2
rq==1.3.0
rq-dashboard==0.3.4
# pin IOOS QC to a particular commit on main until stable version with fixes is released
-e git+https://github.com/ioos/ioos_qc@57813de111af201f8d7b68c7fcd6337bf37dd6ed#egg=ioos_qc
git+https://github.com/ioos/ioos_qc.git@v2.2.0#egg=ioos_qc
Copy link
Member

Choose a reason for hiding this comment

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

Can we use the stable version from PyPI or conda instead of the GitHub release? GH releases are, sadly, not immutable and if by mistake someone edits that, it may cause some really hard to debug breakages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test QARTOD compatibility with ioos_qc v2.2.0 release
3 participants