Skip to content

Document embedding non-JAR dependencies#1777

Merged
Goooler merged 14 commits into
mainfrom
g/20250923/add-more-non-jar-files
Sep 23, 2025
Merged

Document embedding non-JAR dependencies#1777
Goooler merged 14 commits into
mainfrom
g/20250923/add-more-non-jar-files

Conversation

@Goooler

@Goooler Goooler commented Sep 23, 2025

Copy link
Copy Markdown
Member

@Goooler Goooler requested a review from Copilot September 23, 2025 02:55

Copilot AI left a comment

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.

Pull Request Overview

This PR adds documentation and tests for embedding non-JAR dependencies into shadowed JARs. It addresses scenarios where remote dependencies are not JAR files (e.g., POM files, native libraries) that cause build failures when included in standard runtime configurations.

  • Adds functional test for custom configuration handling of non-JAR dependencies
  • Documents solution using custom configurations with from() method to avoid ZIP expansion errors
  • Updates cross-references between documentation sections

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/functionalTest/kotlin/com/github/jengelman/gradle/plugins/shadow/JavaPluginsTest.kt Adds test case verifying non-JAR dependencies can be embedded via custom configuration
docs/configuration/dependencies/README.md Documents new section on embedding non-JAR dependencies with code examples
docs/configuration/README.md Adds cross-reference link to new documentation section

Comment thread docs/configuration/dependencies/README.md Outdated
Comment thread docs/configuration/dependencies/README.md Outdated
Comment thread docs/configuration/dependencies/README.md Outdated
Comment thread docs/configuration/README.md
Goooler and others added 3 commits September 23, 2025 10:56
@Goooler Goooler requested a review from Copilot September 23, 2025 02:57

This comment was marked as outdated.

Goooler and others added 2 commits September 23, 2025 10:59
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Goooler Goooler requested a review from Copilot September 23, 2025 03:01
@Goooler Goooler changed the title Document embedding non-jar dependencies Document embedding non-JAR dependencies Sep 23, 2025

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Goooler Goooler requested a review from Copilot September 23, 2025 03:02

This comment was marked as outdated.

@Goooler Goooler requested a review from Copilot September 23, 2025 03:06

This comment was marked as outdated.

@Goooler Goooler requested a review from Copilot September 23, 2025 03:08

This comment was marked as outdated.

@Goooler Goooler requested a review from Copilot September 23, 2025 03:17

This comment was marked as outdated.

@Goooler Goooler merged commit 2c0e8e6 into main Sep 23, 2025
7 checks passed
@Goooler Goooler deleted the g/20250923/add-more-non-jar-files branch September 23, 2025 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants