95 setup functions#96
Conversation
…ules Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #96 +/- ##
===========================================
- Coverage 79.49% 46.68% -32.82%
===========================================
Files 7 11 +4
Lines 239 407 +168
===========================================
Hits 190 190
- Misses 49 217 +168 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| ) | ||
|
|
||
|
|
||
| def is_relevant(field_list): |
There was a problem hiding this comment.
Just a reminder that is_relevant was refactored in the notebooks.
There was a problem hiding this comment.
Aha, I'm using your PR's version https://github.com/open-contracting/notebooks-ocds/blob/01e19bf8f3749516085c096d674f6cb993229388/component_setup_usability.py#L91
There was a problem hiding this comment.
I think relevant_rules as a global might be better, so it can be reused if needed
|
We'll need something like this in pyproject.toml: [tool.setuptools.package-data]
kingfishercolab = ["indicators/*.json"]Unless we move the JSON to data-support per open-contracting/notebooks-ocds#106 |
|
For the JSON Schema lints, we can edit standard-maintenance-scripts to ignore it, either in ocds_schema_exceptions or excluded_repo_name. Or we can update the shared lint.yml to accept configuration instead of editing those. Edit: You can now exclude files in lint.yml by adding: with:
ocds-schema-exceptions: my-schema.json |
26a1df2 to
5454d69
Compare
Ok, still failing with
|
|
Just this: indicators.schema.json |
|
@yolile Lints now pass. |
Uh oh!
There was an error while loading. Please reload this page.