Skip to content

Support Spring Boot 3.x and Java 17 #33

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 2 commits into
base: master
Choose a base branch
from

Conversation

slgobinath
Copy link

Thanks to @pramoth for this library and @mdillenk for the attempt to migrate to Spring Boot 3.x
The PR #31 works but some changes in the H2 database cause most unit tests to fail. I have fixed the test configurations to get all tests pass successfully.

I'd highly appreciate it if you could release the changes to a new version.

@tnbossi123
Copy link

@pramoth this PR, would be amazing if it was merged. Would save so much time to finding another substitute!

@RahatMinhaj
Copy link

@pramoth could you please merge the request if all the requirements covered

@nicolasfavier
Copy link

nicolasfavier commented Apr 4, 2024

Hello, could you please merge this pull request (or make a release ) ? if you need help I would be glad to help you

@kjellski
Copy link

Really sad that one can't open their project to new contributors in a save way... @pramoth seems to have moved on, but we'd love to use his legacy...

@tnbossi123
Copy link

@slgobinath Could you create a repo with your merged code? So we could use your changes, it does not seem that @pramoth is active longer.

@mdillenk
Copy link

Being under the gun at the time I just built and released the project in our nexus server and had our projects depend on that internal dependency going forward.

@slgobinath
Copy link
Author

Hi @tnbossi123 ,
Here's the repo with the changes https://github.com/slgobinath/specification-with-projection/tree/feature-spring-boot-3 but unfortunately I don't have time to maintain it as a fork.

I also moved to Quarkus with QueryDSL so I no longer depend on this library.

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