Skip to content

Fix maven package assembly in uberjars - #5346

Open
AyanSinhaMahapatra wants to merge 2 commits into
developfrom
maven-package-assembly
Open

Fix maven package assembly in uberjars#5346
AyanSinhaMahapatra wants to merge 2 commits into
developfrom
maven-package-assembly

Conversation

@AyanSinhaMahapatra

@AyanSinhaMahapatra AyanSinhaMahapatra commented Oct 6, 2026

Copy link
Copy Markdown
Member

Fix package assembly in uberjars with multiple pom.xml manifests where we were not assembling package manifests for the original package together.

Reference: aboutcode-org/scancode.io#2248

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

@mstykow mstykow mentioned this pull request Oct 7, 2026
Merged
@AyanSinhaMahapatra
Fix maven package assembly in uberjars
e7b8454 
Fix package assembly in uberjars with multiple pom.xml manifests
where we were not assembling package manifests for the original package
together.

Reference: aboutcode-org/scancode.io#2248
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Update test expectations with pname additions
fec67e0 
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@AyanSinhaMahapatra