Skip to content

Include lombok.config in source release assembly#4808

Merged
hangc0276 merged 1 commit into
apache:masterfrom
void-ptr974:fix/lombok-config-source-release
Jun 1, 2026
Merged

Include lombok.config in source release assembly#4808
hangc0276 merged 1 commit into
apache:masterfrom
void-ptr974:fix/lombok-config-source-release

Conversation

@void-ptr974
Copy link
Copy Markdown
Contributor

@void-ptr974 void-ptr974 commented May 30, 2026

Motivation

The source release archive should include the top-level lombok.config.

Without it, compiling from the source archive fails with:

The @CustomLog is not configured; please set lombok.log.custom.declaration in lombok.config.

Changes

Include lombok.config in the source assembly descriptor.

Verification

  • Confirmed the generated source tarball contains bookkeeper-4.18.0/lombok.config.
  • Built from the regenerated source tarball with mvn -DskipTests -Dexec.skip=true -Dmaven.antrun.skip=true clean install.

Copy link
Copy Markdown
Member

@lhotari lhotari left a comment

Choose a reason for hiding this comment

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

LGTM

@hangc0276 hangc0276 merged commit 3c4a026 into apache:master Jun 1, 2026
34 of 36 checks passed
@hangc0276 hangc0276 added this to the 4.18.0 milestone Jun 1, 2026
hangc0276 pushed a commit that referenced this pull request Jun 1, 2026
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