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

Improve consistency in codegen #9609

Merged
merged 1 commit into from
May 27, 2020

Conversation

ia3andy
Copy link
Contributor

@ia3andy ia3andy commented May 26, 2020

This is part of #8178

The result is just a tiny bit better, this is just a step in my journey. Having more consistency and readability will help in the refactoring process..

  • Use only the values in QuarkusCommandInvocation, because having the Properties and the Values made it confusing
  • Make QuarkusCommandInvocation more consistent by making it partially immutable (only values are mutable) and enforcing non null params in the constructor
  • Remove legacy @deprecated stuff

This is part of quarkusio#8178

The result is just a tiny bit better, this is just a step in my journey. Having more consistency and readability will help in the refactoring process..

- Use only the values in `QuarkusCommandInvocation`, because having the Properties and the Values made it confusing
- Make `QuarkusCommandInvocation` more consistent by making it partially immutable (only values are mutable) and enforcing non null params in the constructor
- Remove legacy @deprecated stuff
@boring-cyborg boring-cyborg bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle area/maven area/platform Issues related to definition and interaction with Quarkus Platform labels May 26, 2020
@ia3andy ia3andy requested a review from aloubyansky May 26, 2020 14:51
@ia3andy
Copy link
Contributor Author

ia3andy commented May 26, 2020

Removing the @Deprecated may break build with code.quarkus.io, but this is a really easy change..

@aloubyansky aloubyansky merged commit cee1b75 into quarkusio:master May 27, 2020
@ia3andy ia3andy deleted the refactor-codegen branch May 27, 2020 06:45
@gsmet gsmet added this to the 1.6.0 - master milestone Jun 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle area/maven area/platform Issues related to definition and interaction with Quarkus Platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants