Skip to content

Sync ShadowApplicationPlugin with ApplicationPlugin#1224

Merged
Goooler merged 7 commits intomainfrom
g/20250205/update-run-task
Feb 5, 2025
Merged

Sync ShadowApplicationPlugin with ApplicationPlugin#1224
Goooler merged 7 commits intomainfrom
g/20250205/update-run-task

Conversation

@Goooler
Copy link
Copy Markdown
Member

@Goooler Goooler commented Feb 5, 2025

bin.filePermissions { it.unix(493) }
bin.filePermissions { it.unix(UNIX_SCRIPT_PERMISSIONS) }
}
distSpec.with(applicationExtension.applicationDistribution)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is a newly added feature.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@Goooler Goooler force-pushed the g/20250205/update-run-task branch from 0823029 to 9279439 Compare February 5, 2025 07:11
@Goooler Goooler changed the title Update addRunTask Sync ShadowApplicationPlugin with ApplicationPlugin Feb 5, 2025
@Goooler Goooler marked this pull request as ready for review February 5, 2025 07:23
@Goooler Goooler requested a review from Copilot February 5, 2025 07:23
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 2 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (3)
  • api/shadow.api: Language not supported
  • src/functionalTest/kotlin/com/github/jengelman/gradle/plugins/shadow/ApplicationPluginTest.kt: Language not supported
  • src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/ShadowApplicationPlugin.kt: Language not supported

@Goooler Goooler enabled auto-merge (squash) February 5, 2025 07:24
@Goooler Goooler merged commit 902f2ae into main Feb 5, 2025
@Goooler Goooler deleted the g/20250205/update-run-task branch February 5, 2025 07:29
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.

Property applicationDistribution not recognized Can't include a directory outside the dist directory into the application distribution

2 participants