Skip to content

3.2 merge dev #4727

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

Merged
merged 23 commits into from
Jun 15, 2025
Merged

3.2 merge dev #4727

merged 23 commits into from
Jun 15, 2025

Conversation

ralfhandl
Copy link
Contributor

This should resolve the merge conflicts between dev and v3.2-dev because it brings in all missing commits from dev and then adds a merge commit. Looks good locally with git merge.

  • schema changes are included in this pull request
  • schema changes are needed for this pull request but not done yet
  • no schema changes are needed for this pull request

handrews and others added 23 commits June 12, 2025 18:28
But don't count standard JSON Schema keywords.
There needs to be a local `$dynamicAnchor`, but it is never
actually evaluated.
Co-authored-by: Ralf Handl <[email protected]>
Co-authored-by: Ralf Handl <[email protected]>
Since we are testing with a placeholder, we need to match
the placeholder.  This will unfortunately need to be different
on each new release line branch, so let's separate this test
case into its own file.
Use full schema (schema-base.yaml) in schema.test.mjs
Use full schema (schema-base.yaml) for coverage
…a-tests

schema test coverage: set exit code only if base branch is not dev
and remove unnecessary code, thanks to @jdesrosiers
main: determine OAS Base Vocab name from meta.yaml for schema test coverage
dev: don't hard-code vocabulary in test code
@ralfhandl ralfhandl requested review from a team as code owners June 15, 2025 11:28
@handrews handrews merged commit 8604a79 into OAI:v3.2-dev Jun 15, 2025
2 checks passed
@ralfhandl ralfhandl deleted the 3.2-merge-dev branch June 15, 2025 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants