json schema validation improvements #7

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.

Sign up for GitHub

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

Jump to bottom

Open

zrouliaa wants to merge 1 commit into Iduoad:main from zrouliaa:zrouliaa/validation

zrouliaa commented May 9, 2025

this pr addresses several issues related to the schema validation script:

what changed

fixed missing quote in regions.json that was causing validation errors
migrated from deprecated refresolver to modern referencing library
improved uri handling with multiple reference patterns for each schema file
added clear error messages for empty files
fixed indentation issues in the main function

benefits

better cross-schema reference resolution
properly handles relative and absolute schema references
more robust validation against empty or malformed files
increased compatibility with latest json schema standards


          validation script improvements and region enum fix

d521394

Owner

Iduoad commented May 10, 2025 •

edited

Loading

Changed the target to main. I merged #6 already. Let's merge this to main directly.

Iduoad changed the base branch from feat/add-validation to main

May 10, 2025 02:02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet