Skip to content

chore: Align json-schema-validator for camel-yaml-dsl tests to what i…#11207

Merged
igarashitm merged 1 commit into
apache:mainfrom
igarashitm:networknt-validator
Aug 25, 2023
Merged

chore: Align json-schema-validator for camel-yaml-dsl tests to what i…#11207
igarashitm merged 1 commit into
apache:mainfrom
igarashitm:networknt-validator

Conversation

@igarashitm
Copy link
Copy Markdown
Member

…s used for camel-json-validator component

Description

camel-yaml-dsl validation tests use java-json-tools/json-schema-validator which stopped the development in 2020 with JSON schema draft 4. Let's migrate it to networknt/json-schema-validator which is already used for camel-json-validator component and supports newer drafts.

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

@igarashitm igarashitm requested a review from lburgazzoli August 25, 2023 17:17
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🐫 Maintainers, please note that first-time contributors require manual approval for the GitHub Actions to run.

⚠️ Please note that the changes on this PR may be tested automatically if they change components.

🤖 Use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@igarashitm igarashitm merged commit d985d7c into apache:main Aug 25, 2023
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