Skip to content

updates from main - #157

Merged
bart1 merged 30 commits into
133-fail-early-on-not-finding-a-dutch-converterfrom
mergefix
Dec 2, 2025
Merged

bart1 merged 30 commits into
133-fail-early-on-not-finding-a-dutch-converterfrom
mergefix

Conversation

@bart1

@bart1 bart1 commented Dec 2, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Bart and others added 29 commits August 14, 2025 14:00
@bart1 this test is failing (returns NULL rather than the radar name). Rather than fixing it, I don't see the need for this test. We don't use the catch_cnd() approach anywhere else
Fix typo + make error messages consistent
Comment thread R/get_pvol_nl.R Resolved
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@bart1
bart1 marked this pull request as ready for review December 2, 2025 19:58
@bart1
bart1 merged commit d1414a2 into 133-fail-early-on-not-finding-a-dutch-converter Dec 2, 2025
1 of 4 checks passed
@bart1
bart1 deleted the mergefix branch December 2, 2025 19:58
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