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

chore: update to latest Quarkus micro #1077

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

gsmet
Copy link
Contributor

@gsmet gsmet commented Dec 23, 2024

While I agree you should stay compatible with the minor requested by the Operator SDK extension, I don't think it's a good idea to stay on the micro as you are missing all the fixes we are pushing since at the moment the version used here is 3.17.0.

So my advice is: update to the latest micro of the minor branch requested for the Operator SDK.

This is important for #1076 as otherwise, you will have to wait a lot longer to get the fix. Also it's good practice in general.

While I agree you should stay compatible with the minor requested by the
Operator SDK extension, I don't think it's a good idea to stay on the
micro as you are missing all the fixes we are pushing since at the
moment the version used here is 3.17.0.

So my advice is: update to the latest micro of the minor branch
requested for the Operator SDK.

This is important for project-ncl#1076 as otherwise, you will have to wait a lot
longer to get the fix. Also it's good practice in general.
@gsmet gsmet changed the title Update to latest Quarkus micro chore: update to latest Quarkus micro Dec 24, 2024
@goldmann
Copy link
Contributor

goldmann commented Jan 8, 2025

Thanks a lot. I think it makes sense. We will use this approach from now on.

/cc @vibe13 @janinko

@goldmann goldmann merged commit e096e0c into project-ncl:main Jan 8, 2025
9 checks 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