Skip to content
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

Drop jhipster-dependencies dependency. #25565

Closed
1 task
mshima opened this issue Mar 20, 2024 · 8 comments · Fixed by #25602
Closed
1 task

Drop jhipster-dependencies dependency. #25565

mshima opened this issue Mar 20, 2024 · 8 comments · Fixed by #25602

Comments

@mshima
Copy link
Member

mshima commented Mar 20, 2024

Overview of the feature request

Drop jhipster-dependencies dependency and use spring-boot's dependency management.
jhipster-framework will be kept.

Motivation for or Use Case

The idea of jhispter-dependencies is to add a centralized dependency management for jhipster project to unify and simplify dependencies.
Currently we have a mixed of jhispter-dependencies and generator-jhipster managed dependencies, so the purpose is only partially met.

Removing jhispter-dependencies will provide:

  • less maintenance burden.
  • easier release process (generator-jhipster only, jhipster-framework when needed).
  • unified repository for dependency versions.
  • better CI due to selective built-in generator-jhipster managed dependencies.
  • spring-boot standard built tool implementation.
Related issues or PR
  • Checking this box is mandatory (this is just to show you read everything)
@mshima
Copy link
Member Author

mshima commented Mar 20, 2024

What do you think @mraible @DanielFran @deepu105?

@mraible
Copy link
Contributor

mraible commented Mar 20, 2024

Does this mean we can use spring-boot-starter-parent as a parent in generated projects? I remember talking to Josh Long a few years ago and he was appalled that JHipster managed its own dependencies rather than relying on Spring Boot's parent.

@mshima
Copy link
Member Author

mshima commented Mar 20, 2024

Does this mean we can use spring-boot-starter-parent as a parent in generated projects?

We probably can use spring-boot-dependencies only.
I think the benefit of spring-boot-starter-parent is to avoid dependency conflicts outside <dependencies>.

I remember talking to Josh Long a few years ago and he was appalled that JHipster managed its own dependencies rather than relying on Spring Boot's parent.

We will be still managing our own dependencies, but at generator-jhipster side only.

@mshima
Copy link
Member Author

mshima commented Mar 20, 2024

We are managing on generator-jhipster side:

And at jhipster-dependencies side:

Only springdoc-openapi dependency in jhipster-dependencies is used by the application jhipster --defaults.

@DanielFran
Copy link
Member

I am in favour too!
Since we found a different behaviour with the bom in last maven release, it is better to manage the dependencies in the generator.

@mshima
Copy link
Member Author

mshima commented Mar 20, 2024

Indeed we define a custom testcontainers version.
But the version from spring-boot-dependencies is used anyway

@atomfrede
Copy link
Member

Less overhead for releases and being closes to upstream boot would be a beneficial imho. So lets do it.

@deepu105
Copy link
Member

deepu105 commented Mar 20, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants