Skip to content

Adopt the lombok@1.18.27 edge release#3031

Merged
testforstephen merged 1 commit into
redhat-developer:masterfrom
testforstephen:jinbo_lombok
Mar 30, 2023
Merged

Adopt the lombok@1.18.27 edge release#3031
testforstephen merged 1 commit into
redhat-developer:masterfrom
testforstephen:jinbo_lombok

Conversation

@testforstephen

Copy link
Copy Markdown
Collaborator

See projectlombok/lombok#3332 (comment), this fixes a significant lombok regression #2887.

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
@testforstephen testforstephen linked an issue Mar 30, 2023 that may be closed by this pull request
Comment thread gulpfile.js
.on("error", reject)
.on('end', () => {
fse.renameSync("./lombok/lombok-edge.jar", "./lombok/lombok-1.18.27.jar");
resolve();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

just wondering if we can simply put done() here without adding new promise/async/await stuffs?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Anyway, just forget about it, it doesn't matter.

@testforstephen
testforstephen merged commit f43dc66 into redhat-developer:master Mar 30, 2023
@testforstephen
testforstephen deleted the jinbo_lombok branch March 31, 2023 01:54
gayanper pushed a commit to gayanper/vscode-java that referenced this pull request May 27, 2023
Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
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.

lombok.config ignored in 1.14.0

3 participants