Skip to content

Migrate spring boot dependencies to version catalog #4441

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

Open
wants to merge 7 commits into
base: ryan/version-catalog-9
Choose a base branch
from

Conversation

rbro112
Copy link
Member

@rbro112 rbro112 commented May 22, 2025

📜 Description

Version catalog adoption

#skip-changelog

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

@rbro112 rbro112 changed the title Ryan/version catalog 10 Migrate spring boot dependencies to version catalog May 22, 2025
val springBoot3StarterJdbc = "org.springframework.boot:spring-boot-starter-jdbc:$springBoot3Version"
val springBoot3StarterActuator = "org.springframework.boot:spring-boot-starter-actuator:$springBoot3Version"
val springBoot3StarterOpenTelemetry = "io.opentelemetry.instrumentation:opentelemetry-spring-boot-starter:${OpenTelemetry.otelInstrumentationVersion}"
// val springBootStarter = "org.springframework.boot:spring-boot-starter:$springBootVersion"
Copy link
Contributor

Choose a reason for hiding this comment

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

don’t forget to remove this before merging!

spring-boot-two = { id = "org.springframework.boot", version.ref = "springTwo" }
spring-boot-three = { id = "org.springframework.boot", version.ref = "springThree" }
springboot2 = { id = "org.springframework.boot", version.ref = "springboot2" }
springboot3 = { id = "org.springframework.boot", version.ref = "springboot2" }
Copy link
Member

Choose a reason for hiding this comment

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

Should this be?

Suggested change
springboot3 = { id = "org.springframework.boot", version.ref = "springboot2" }
springboot3 = { id = "org.springframework.boot", version.ref = "springboot3" }

@rbro112 rbro112 force-pushed the ryan/version-catalog-10 branch 2 times, most recently from 63fff7d to a25a54e Compare May 23, 2025 17:50
lbloder and others added 6 commits May 23, 2025 10:50
* Bump Spring Boot to 3.4.1

* changelog

* update to spring boot 3.4.2

* fix jdbc dependencies for jakarta samples

* update readme

* update logback dependency in jakarta samples

* [WIP] test with spring boot 3.5.0

* Update buildSrc/src/main/java/Config.kt

* adapt to breaking change in spring-graphql-1.4.0, fix test, add withOptions to sample

* bump to Spring Boot 3.5.0

* trigger CI?

* update spring boot version in toml

* move changelog

* revert maven milestone repos

* fix changelog

* restore old changelog entry

* fix gh pr link

---------

Co-authored-by: Alexander Dinauer <[email protected]>
Co-authored-by: Alexander Dinauer <[email protected]>
Co-authored-by: Lorenzo Cian <[email protected]>
…ead of `OtelSentryPropagator` (#4434)

* Fork isolation scope in SpanProcessor

* improve creator strings

* changelog

* remove irrelevant tests
@rbro112 rbro112 force-pushed the ryan/version-catalog-10 branch from a25a54e to 77f5f1a Compare May 23, 2025 17:50
@rbro112 rbro112 changed the base branch from main to ryan/version-catalog-9 May 23, 2025 17:51
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.

6 participants