-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
ArC - Switch to @ConfigMapping #46064
Conversation
Thanks for your pull request! Your pull request does not follow our editorial rules. Could you have a look?
This message is automatically generated by a bot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I don't think anyone outside of ArC uses these classes, so this should be fine compatibility-wise. |
Status for workflow
|
🎊 PR Preview 3778750 has been successfully built and deployed to https://quarkus-pr-main-46064-preview.surge.sh/version/main/guides/
|
Let's wait for @mkouba. He's on PTO. |
Status for workflow
|
We have to wait for Qute anyway and I prefer giving @mkouba a chance to check things. |
Part of #45446