-
Notifications
You must be signed in to change notification settings - Fork 1.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
User lombok Getter and Setter. #5658
Conversation
It seems lombok generates `get` for Boolean instead of `is`.
liquibase-standard/src/main/java/liquibase/sql/CallableSql.java
Dismissed
Show dismissed
Hide dismissed
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.
Approved.
This looks ok to me. If you are planning to keep doing these changes for some other files I would recommend you to go in some specific order, maybe in an upcoming PR to keep adding Lombok annotations to the remaining liquibase-standard
module files, and once finish with this module (which is the bigger) then continue with others.
Thanks,
Daniel.
This is a step to use lombok Getter and Setter, as hinted in another PR.
Not all potential changes are done, because the PR would be huge. Please advise if this is fine, or project-wide changes should also be added.