Skip to content

Allow opt-in impl-to-impl dependencies#200

Merged
JessalynWang merged 1 commit into
amzn:mainfrom
vRallev:ralf/allow-impl-to-impl-dependencies
Jun 15, 2026
Merged

Allow opt-in impl-to-impl dependencies#200
JessalynWang merged 1 commit into
amzn:mainfrom
vRallev:ralf/allow-impl-to-impl-dependencies

Conversation

@vRallev

@vRallev vRallev commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Add nested enableModuleStructure options with allowLibraryImplToImplDependencies, preserving the existing Boolean overload and a false default. This lets sibling implementation variants reuse code without forcing an otherwise-empty :internal module.

Thread the option through the cacheable dependency checker while keeping cross-library and external implementation dependencies forbidden and retaining existing internal-module semantics. Update the public API snapshot and consumer documentation for both Groovy and Kotlin DSL usage.

Add nested `enableModuleStructure` options with `allowImplToImplDependencies`, preserving the existing Boolean overload and a false default. This lets sibling implementation variants reuse code without forcing an otherwise-empty `:internal` module.

Thread the option through the cacheable dependency checker while keeping cross-library and external implementation dependencies forbidden and retaining existing internal-module semantics. Update the public API snapshot and consumer documentation for both Groovy and Kotlin DSL usage.
@JessalynWang JessalynWang merged commit 21b76e1 into amzn:main Jun 15, 2026
20 checks passed
@vRallev vRallev deleted the ralf/allow-impl-to-impl-dependencies branch June 24, 2026 21:46
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.

2 participants