Skip to content

Fix ShadowJar not being successful after includes or excludes are changed#1200

Merged
Goooler merged 7 commits intomainfrom
g/20250126/fix-exclude-include-inputs
Jan 26, 2025
Merged

Fix ShadowJar not being successful after includes or excludes are changed#1200
Goooler merged 7 commits intomainfrom
g/20250126/fix-exclude-include-inputs

Conversation

@Goooler
Copy link
Copy Markdown
Member

@Goooler Goooler commented Jan 26, 2025

@Goooler Goooler changed the title Rerun ShadowJar after excludes or includes changed Fix ShadowJar not being success after includes or excludes are changed Jan 26, 2025
@Goooler Goooler requested a review from Copilot January 26, 2025 11:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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

Files not reviewed (2)
  • src/functionalTest/kotlin/com/github/jengelman/gradle/plugins/shadow/caching/ShadowJarCachingTest.kt: Language not supported
  • src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/tasks/ShadowJar.kt: Language not supported

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Goooler Goooler changed the title Fix ShadowJar not being success after includes or excludes are changed Fix ShadowJar not being successful after includes or excludes are changed Jan 26, 2025
@Goooler Goooler marked this pull request as ready for review January 26, 2025 11:42
@Goooler Goooler force-pushed the g/20250126/fix-exclude-include-inputs branch from e178acb to 7f1db98 Compare January 26, 2025 11:49
@Goooler Goooler enabled auto-merge (squash) January 26, 2025 11:50
@Goooler Goooler merged commit 450b831 into main Jan 26, 2025
@Goooler Goooler deleted the g/20250126/fix-exclude-include-inputs branch January 26, 2025 11:55
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.

ShadowJar task input fingerprint uses configured 'excludes' shadowJar task reports UP-TO-DATE with normal exclude.

2 participants