Merge develop into infrahub-develop - #1370
Merged
Merged
Conversation
…1357) * feat(schema): expose location, value and reason on validation error details Every entry of `SchemaValidationResult.errors` now carries `loc` (the field location as a tuple of keys and indexes), `input` (the value received there) and `reason` (the wording alone), next to the unchanged `field` and `message`, which are rendered from these parts by a single helper. A consumer can report or forward each violation in a structured form without parsing the message text. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * test(schema): assert field is the dotted rendering of loc on a concrete error The parametrized check compared field and loc by rewriting brackets into dots, which hid the rule it was testing. A dedicated test now shows one nested error with its location tuple and rendered path side by side. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * test(schema): check field against loc for every rejected case Drops a test that repeated the unknown-field detail test, asserts the rendered field on the out-of-enum detail test, and restores the field/loc consistency check across all unknown-field and out-of-enum cases through a small helper that spells out the expected rendering. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
Merge stable into develop
Deploying infrahub-sdk-python with
|
| Latest commit: |
ff707a8
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://1448a1ef.infrahub-sdk-python.pages.dev |
| Branch Preview URL: | https://develop.infrahub-sdk-python.pages.dev |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## infrahub-develop #1370 +/- ##
=================================================
Coverage 86.28% 86.28%
=================================================
Files 149 149
Lines 14537 14541 +4
Branches 1995 1994 -1
=================================================
+ Hits 12543 12547 +4
Misses 1431 1431
Partials 563 563
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
infrahub-github-bot-app
Bot
merged commit Sep 17, 2026
68f742b
into
infrahub-develop
46 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merging develop into infrahub-develop after merging pull request #1369.
Summary by cubic
Merges
developintoinfrahub-develop, bringing the structured schema validation errors. Schema validation errors previously exposed onlyfieldandmessage; they now also carryloc,input, andreasonas separate fields, so consumers can report violations without parsing the message text. For missing fields,inputis the enclosing object.Written for commit ff707a8. Summary will update on new commits.