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

Remove boilerplate #132

Closed
wants to merge 4 commits into from
Closed

Conversation

tha0pham
Copy link

@tha0pham tha0pham commented Oct 19, 2020

#126 Remove boilerplate getters, setters, and exhaustive constructors using Lombok

@tha0pham tha0pham closed this Oct 19, 2020
@tha0pham tha0pham reopened this Oct 19, 2020
@denizkilic
Copy link
Contributor

Hi @theopham1816, thanks for adding Lombok dependency to the project. However, this change can break backward compatibility. Also, 111 file changes are a lot. I need to test everything very well to make sure these changes don’t break anything. Let’s keep this PR open for a while.

@mehmetminanc
Copy link

Hey @thaopham1816, thanks for the great job.
@denizkilic I agree that this might introduce unforeseen issues. We can schedule it for the next major release. I can lend a hand with setting up backward compatibility and integration tests.

@denizkilic
Copy link
Contributor

I am going to close this PR because there is no plan to use Lombok soon.

@denizkilic denizkilic closed this Apr 4, 2022
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.

3 participants