Skip to content

deps: upgrade to latest where possible - #2757

Merged
triceo merged 7 commits into
TimefoldAI:1.xfrom
triceo:deps/1.x-quarkus-3.40
Oct 15, 2026
Merged

triceo merged 7 commits into
TimefoldAI:1.xfrom
triceo:deps/1.x-quarkus-3.40

Conversation

@triceo

@triceo triceo commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Align ASM (9.10.1) and JUnit (6.1.3) with the Quarkus BOM.
Yasson moved to the org.eclipse.yasson groupId.
jaxb2-maven-plugin 4.2.0 no longer writes whitespace-only lines into the generated XSDs.
Use the core rewrite-bom 8.90.4 instead of rewrite-recipe-bom, and rewrite-maven-plugin 6.46.1.
Backport of 5d7eb41 from main.
@triceo
triceo requested review from a team and a balanced review from Copilot October 15, 2026 10:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The cross-module dependency upgrades and serialization workaround need human confirmation of integration and native-image compatibility.

0 open findings

What changed in this PR

Updates Timefold Solver’s dependencies and build tooling, with corresponding Quarkus integration and documentation adjustments.

Changes:

  • Upgrades libraries and Maven plugins; switches to the OpenRewrite core BOM.
  • Migrates Quarkus test dependencies and adds a Jackson serialization workaround.
  • Updates quickstarts, Dev UI expectations, and schema whitespace.
File Description
quarkus-integration/​quarkus/​reflection-integration-test/​pom.xml Renames Quarkus test dependency.
quarkus-integration/​quarkus/​devui-integration-test/​src/​test/​java/​ai/​timefold/​solver/​quarkus/​it/​devui/​TimefoldDevUITest.java Expects an empty termination element.
quarkus-integration/​quarkus/​devui-integration-test/​pom.xml Renames Quarkus test dependencies.
quarkus-integration/​quarkus/​deployment/​pom.xml Renames internal test dependency.
quarkus-integration/​quarkus-jsonb/​integration-test/​pom.xml Renames Quarkus test dependency.
quarkus-integration/​quarkus-jackson/​integration-test/​src/​main/​resources/​application.properties Disables reflection-free serializers as a workaround.
quarkus-integration/​quarkus-jackson/​integration-test/​pom.xml Renames Quarkus test dependency.
quarkus-integration/​quarkus-benchmark/​integration-test/​pom.xml Renames Quarkus test dependency.
quarkus-integration/​quarkus-benchmark/​deployment/​pom.xml Renames internal test dependency.
pom.xml Upgrades OpenRewrite Maven plugin.
persistence/​jsonb/​pom.xml Updates Yasson group ID.
docs/​src/​modules/​ROOT/​pages/​quickstart/​quarkus/​quarkus-quickstart.adoc Updates Maven and Gradle test dependencies.
docs/​src/​modules/​ROOT/​pages/​quickstart/​quarkus-vehicle-routing/​quarkus-vehicle-routing-quickstart.adoc Updates Maven test dependency.
core/​src/​main/​resources/​solver.xsd Removes blank lines without schema changes.
build/​build-parent/​pom.xml Upgrades dependencies/plugins and replaces OpenRewrite BOM.
benchmark/​src/​main/​resources/​benchmark.xsd Adjusts whitespace without schema changes.
.github/​dependabot.yml Groups OpenRewrite core updates.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@zepfred
zepfred self-requested a review October 15, 2026 10:58

@zepfred zepfred left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@triceo triceo changed the title chore: upgrade deps deps: upgrade to latest where possible Oct 15, 2026
@triceo
triceo merged commit 311cef7 into TimefoldAI:1.x Oct 15, 2026
31 of 42 checks passed
@triceo
triceo deleted the deps/1.x-quarkus-3.40 branch October 15, 2026 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants