Skip to content

Conversation

@holly-cummins
Copy link
Collaborator

This is useful for own demos, but should also be useful for people trying out the project.

@edeandrea
Copy link
Collaborator

What about all the tests?

@holly-cummins holly-cummins force-pushed the spring-conversion-instructions branch from e0b49ab to 30913d7 Compare November 5, 2025 16:16
@holly-cummins
Copy link
Collaborator Author

What about all the tests?

For the performance measurements, the tests not strictly necessary, and I made the assumption that people wouldn't want to take the time to do the conversion live. So I removed the tests, and that's an opener to a conversation about how we don't have that totally seamless test-conversion experience yet.

But we could certainly add instructions for converting them to this document in a follow-on PR if it's small enough to do live. Looking at the diff, it looks pretty complex, but maybe there's an easy trick? If so, it would be great to document it (so we'd have two options, one is to delete the tests to get stuff buildable and stress-able, and the other is the more involved conversion).

@edeandrea
Copy link
Collaborator

For the performance measurements, the tests not strictly necessary

I agree, if thats the goal.

However, if the goal of the end user is to understand how to get from springboot3 -> quarkus3-with-compatibility (which is what I do live all the time), then we should have instructions.

We can certainly add them in a follow-up, and I can take ownership of it if you'd like (but will most likely be a few weeks before I get to it).

Would you mind creating a separate issue for that?

@holly-cummins
Copy link
Collaborator Author

holly-cummins commented Nov 5, 2025

I've raised #34 (and yes please on the ownership, @edeandrea, I'd have no idea what the steps are!)

@edeandrea edeandrea merged commit b6dccb5 into quarkusio:main Nov 5, 2025
1 check passed
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.

2 participants