Skip to content

Fix regular checks#164

Merged
peterdesmet merged 7 commits intomainfrom
fix_regular_checks
Jan 14, 2026
Merged

Fix regular checks#164
peterdesmet merged 7 commits intomainfrom
fix_regular_checks

Conversation

@bart1
Copy link
Copy Markdown
Collaborator

@bart1 bart1 commented Jan 8, 2026

Some errors in checking that cause failure

@bart1
Copy link
Copy Markdown
Collaborator Author

bart1 commented Jan 8, 2026

Part of the error seems to be an outage at NCEI https://www.ncei.noaa.gov/

Please note: Due to a system outage, NCEI Users should expect delays accessing many datasets. We are working to resolve these issues as soon as possible. We apologize for any inconvenience.

Lets see if it resolves

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.72%. Comparing base (8f780ca) to head (3840f33).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #164   +/-   ##
=======================================
  Coverage   93.72%   93.72%           
=======================================
  Files          24       24           
  Lines        1801     1801           
=======================================
  Hits         1688     1688           
  Misses        113      113           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bart1 bart1 requested a review from peterdesmet January 13, 2026 19:12
@bart1
Copy link
Copy Markdown
Collaborator Author

bart1 commented Jan 13, 2026

I added a api key for the Netherlands and fixed some tests

@peterdesmet peterdesmet merged commit 70bea5f into main Jan 14, 2026
16 of 17 checks passed
@peterdesmet peterdesmet deleted the fix_regular_checks branch January 14, 2026 10:45
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