Repository navigation
ci: run Main on pushes to stage and Integration on PRs into stage - #223
Merged
Merged
Conversation
- register_schema only rewrote refs reachable via the $..allOf jsonpath - a nested model field without Field() metadata emits a bare $ref - those were left dangling once definitions was deleted - keep definitions when a local ref cannot be resolved - closes #101
Contributor
Release previewNo version bump from the current commits (stays at Changelog preview (truncated)Preview via python-semantic-release and conventional commits. |
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
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.
Prepares
stageas the collection branch for the reviewed pull requests that go tomaintogether.Changes
.github/workflows/integration.yml: thepull_requesttrigger now includesstage, so every pull request intostageruns the integration tests against the live test wiki. Comment updated..github/workflows/main.yml: thepushtrigger now includesstage, so lint and unit tests run on the merged state ofstageafter each merge. This shows conflicts between pull requests that pass on their own.docs/dev.md: the integration test section namesstageas well.Rationale
stageran the Main workflow (andvalidate-codecov-configwhen it changedcodecov.yaml), but not Integration. Integration ran automatically only on pull requests intomain.main, so a merge intostagepublishes nothing.main, so merging it also bringsstageup to date withmain.Tests
make checkandmake testpass locally (802 passed).