Skip to content

Decouple R8 from minimization with a deprecated compatibility DSL#2123

Open
Goooler wants to merge 2 commits into
mainfrom
g/20260724/explicit-r8-block
Open

Decouple R8 from minimization with a deprecated compatibility DSL#2123
Goooler wants to merge 2 commits into
mainfrom
g/20260724/explicit-r8-block

Conversation

@Goooler

@Goooler Goooler commented Jul 24, 2026

Copy link
Copy Markdown
Member

Follow up #2077 (comment).


  • CHANGELOG's "Unreleased" section has been updated, if applicable.

Comment thread src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/tasks/ShadowJar.kt Outdated
@Goooler
Goooler force-pushed the g/20260724/explicit-r8-block branch 2 times, most recently from 035281f to 1f73574 Compare July 24, 2026 04:07
@Goooler
Goooler requested a review from Copilot July 24, 2026 04:12

This comment was marked as outdated.

@Goooler
Goooler force-pushed the g/20260724/explicit-r8-block branch from 1f73574 to 687ca72 Compare July 24, 2026 04:17
@Goooler
Goooler requested a review from Copilot July 24, 2026 04:18

This comment was marked as outdated.

@Goooler
Goooler force-pushed the g/20260724/explicit-r8-block branch 6 times, most recently from 2acd1dc to a473baf Compare July 24, 2026 04:50
@Goooler Goooler changed the title Decouple R8 from minimization Decouple R8 from minimization and deprecate the old DSL Jul 24, 2026
@Goooler
Goooler force-pushed the g/20260724/explicit-r8-block branch from a473baf to b32b55b Compare July 24, 2026 04:59
@Goooler Goooler changed the title Decouple R8 from minimization and deprecate the old DSL Decouple R8 from minimization with a deprecated compatibility DSL Jul 24, 2026
@Goooler
Goooler force-pushed the g/20260724/explicit-r8-block branch 2 times, most recently from c24af26 to 24712f9 Compare July 24, 2026 09:33
@Goooler
Goooler force-pushed the g/20260724/explicit-r8-block branch from 24712f9 to 8452ee2 Compare July 24, 2026 09:36

@ZacSweers ZacSweers 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.

I don't really see the value in this as it creates two different minimizing blocks rather than just one that supports different impls

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.

3 participants