Skip to content

Conversation

@felixcremer
Copy link
Member

This adds a check for the type key in the Catalog construction if it is available.
This also adds a pretty printing for OrderedDictWrapper to make it lazy.

I also added a function to get the search url because this should be provided as an entry in the JSON file with some metadata.

For the Catalog check we test whether the type key is available because according to #22 (comment) this is not always provided.

We have to make the check whether the type is available because some examples don't provide it. But if a type key is available it should be Catalog.
@felixcremer
Copy link
Member Author

This would close #24

@codecov
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 88.88889% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.77%. Comparing base (411db70) to head (1d6519d).
⚠️ Report is 14 commits behind head on main.

Files with missing lines Patch % Lines
src/utils.jl 84.21% 3 Missing ⚠️
src/catalog.jl 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
+ Coverage   73.07%   75.77%   +2.69%     
==========================================
  Files           9        9              
  Lines         286      322      +36     
==========================================
+ Hits          209      244      +35     
- Misses         77       78       +1     

☔ 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.

@Alexander-Barth
Copy link
Member

Thanks a lot Felix!

@Alexander-Barth Alexander-Barth merged commit e2f64b6 into JuliaClimate:main Aug 4, 2025
6 checks passed
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