-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: simplify React support #432
Conversation
Format Checker ReportAll files are correctly formatted |
5991570
to
dc95482
Compare
...resources/com/github/mcollovati/quarkus/hilla/deployment/crud/panache/application.properties
Show resolved
Hide resolved
runtime-react/src/main/codestarts/quarkus/quakus-hilla-maven-codestart/base/README.tpl.qute.md
Outdated
Show resolved
Hide resolved
This change introduces the quarkus-hilla-react extension, to be used for application using React as front-end development framework. In addition, it also updates the codestarts to provide a working application, leveraging on Hilla scaffold utility. Fixes #393
Co-authored-by: Dario Götze <Dario@ok.de>
805e3c3
to
ca23046
Compare
causes infinite loop at runtime when Quarkus resolves dependencies
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.
I would like to discuss the thing with the naming of the artifacts.
My suggestion was to add the react prefix simply to everything and not in between. So that simply by looking at the prefix it is clear, what it is targeting. I see the reason, why it is in between, because it is hilla-react
targeting. How is hilla doing the naming of maven artifacts with the react thing?
...github/mcollovati/quarkus/hilla/deployment/SpringDiExtensionSupportReactiveEndpointTest.java
Show resolved
Hide resolved
...rc/test/resources/com/github/mcollovati/quarkus/hilla/deployment/test-application.properties
Outdated
Show resolved
Hide resolved
...rc/test/resources/com/github/mcollovati/quarkus/hilla/deployment/test-application.properties
Outdated
Show resolved
Hide resolved
...sources/com/github/mcollovati/quarkus/hilla/deployment/test-spring-di-application.properties
Outdated
Show resolved
Hide resolved
Hilla has Could you please show me the naming you are thinking about, by providing examples for all our artifacts? |
See the comments on the code, where I switched the 'react' tag to the end |
Co-authored-by: Dario Götze <Dario@ok.de>
This change introduces the quarkus-hilla-react extension, to be used for application using React as front-end development framework. In addition, it also updates the codestarts to provide a working application, leveraging on Hilla scaffold utility.
Fixes #393