Skip to content

✨ Schema validation #1441

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

✨ Schema validation #1441

wants to merge 1 commit into from

Conversation

ubmarco
Copy link
Member

@ubmarco ubmarco commented Apr 16, 2025

Experimental implementation of a schema validation, aka. ontology check.
This is based on jsonschema, but also supports network validations.

@ubmarco ubmarco changed the title ✨ Schema validation (WIP) ✨ Schema validation 🚧 Apr 17, 2025
@ubmarco ubmarco mentioned this pull request Apr 22, 2025
@ubmarco ubmarco changed the title ✨ Schema validation 🚧 ✨ Schema validation Apr 24, 2025
@ubmarco ubmarco force-pushed the mh-schema-validation branch 6 times, most recently from 5cf6b49 to 75c6211 Compare May 2, 2025 10:55
@ubmarco ubmarco force-pushed the mh-schema-validation branch 2 times, most recently from 61538a3 to 22130fc Compare May 15, 2025 10:29
chrisjsewell added a commit that referenced this pull request May 15, 2025
This fixes the problem, that the `option_spec` attributes are modified
on a class level for:

- `NeedDirective`
- `NeedextendDirective`
- `NeedserviceDirective`

and thus remain changed for subsequent sphinx runs in the same
interpreter session.

The problem was a finding of
#1441.

This is particularly of issue for SN test cases, which all run in the
same Python interpreter,
and should ideally be reproducible, independent of any preceding tests
run.

The PR defines a `reset_options_spec` class method for modified
directives,
to revert the runtime changes during the `config-inited` event.

---------

Co-authored-by: Chris Sewell <chrisj_sewell@hotmail.com>
@ubmarco ubmarco force-pushed the mh-schema-validation branch from 22130fc to a8a8eca Compare May 15, 2025 15:41
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.

1 participant