Skip to content

bump lombok from 1.18.20 to 1.18.22 to support java17 compile#3097

Merged
dlg99 merged 1 commit intoapache:masterfrom
hezhangjian:bump-lombok-1-18-22
Mar 15, 2022
Merged

bump lombok from 1.18.20 to 1.18.22 to support java17 compile#3097
dlg99 merged 1 commit intoapache:masterfrom
hezhangjian:bump-lombok-1-18-22

Conversation

@hezhangjian
Copy link
Copy Markdown
Member

@hezhangjian hezhangjian commented Mar 12, 2022

Motivation

Copy link
Copy Markdown
Contributor

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I started the same kind of work here
#3053

@hezhangjian
Copy link
Copy Markdown
Member Author

rerun failure checks

@hezhangjian
Copy link
Copy Markdown
Member Author

@nicoloboschi Thanks for your attention and great work. I was trying to test #3091 on my laptop. I think we can merge this PR first to let people can run unittest with jdk17.
cc @dlg99 @eolivelli

@dlg99 dlg99 added this to the 4.15.0 milestone Mar 15, 2022
@dlg99 dlg99 merged commit 9d2d827 into apache:master Mar 15, 2022
@dlg99
Copy link
Copy Markdown
Contributor

dlg99 commented Mar 15, 2022

@shoothzj merged

@hezhangjian hezhangjian deleted the bump-lombok-1-18-22 branch June 17, 2022 06:32
Ghatage pushed a commit to sijie/bookkeeper that referenced this pull request Jul 12, 2024
### Motivation
- required for compilation on JDK 17
- see https://projectlombok.org/changelog

Reviewers: Andrey Yegorov <None>, Nicolò Boschi <boschi1997@gmail.com>

This closes apache#3097 from Shoothzj/bump-lombok-1-18-22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants