Skip to content

Remove redundant stdlib declarations and module gitignores - #346

Merged
fornewid merged 1 commit into
developfrom
remove-redundant-declarations
Oct 4, 2026
Merged

fornewid merged 1 commit into
developfrom
remove-redundant-declarations

Conversation

@fornewid

@fornewid fornewid commented Oct 4, 2026

Copy link
Copy Markdown
Member

Removes declarations that other mechanisms already cover.

  • kotlin-stdlib: the Kotlin Gradle plugin adds kotlin-stdlib to every Kotlin module (kotlin.stdlib.default.dependency isn't disabled), so the 28 explicit implementation libs.kotlin.stdlib lines and the kotlin-stdlib catalog entry go. Two dependencies blocks that only held that line are removed.
  • .gitignore: the root .gitignore already ignores build/, so the 35 module .gitignore files that only contained /build are deleted. app/.gitignore's /release moves to the root as /app/release/; it's anchored so that the tracked app/src/release/ stays tracked.

dependencyGuard baseline

kotlin-stdlib-jdk7 and kotlin-stdlib-jdk8 now resolve to 1.8.21 instead of 2.2.21. The explicit kotlin-stdlib-jdk8:2.2.21 declaration used to lift them; now they come from okhttp 4.12.0. This is expected and harmless:

  • Since Kotlin 1.8 both artifacts are empty. The 1.8.21 and 2.2.21 jars contain only MANIFEST.MF and META-INF/versions/9/module-info.class.
  • kotlin-stdlib's Gradle metadata only requires them to be at least 1.8.0.
  • :app:checkDebugDuplicateClasses and :app:checkReleaseDuplicateClasses pass.

Verification

  • All Android modules compile (debug main, unit test and androidTest), and so do the JVM modules.
  • ./gradlew dependencyGuard manifestShield proguardShieldFast moduleRules passes. The proguardShield, manifestShield and module-rules baselines are unchanged.
  • git check-ignore still ignores every module's build/ and app/release/ outputs, and git status shows no new untracked files.

πŸ€– Generated with Claude Code

https://claude.ai/code/session_01Upxj1FvQeDpycupKdPQHUZ

The Kotlin Gradle plugin adds kotlin-stdlib to every Kotlin module, so
drop the 28 explicit kotlin-stdlib-jdk8 declarations and their catalog
entry. kotlin-stdlib-jdk7/jdk8 now resolve to 1.8.21, requested by
okhttp 4.12.0, instead of being lifted to 2.2.21. Since Kotlin 1.8 both
are empty artifacts, and kotlin-stdlib only requires them to be at
least 1.8.0.

The root .gitignore already ignores build/, so delete the 35 module
.gitignore files that only ignored /build, and move app's /release rule
to the root as /app/release/.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Upxj1FvQeDpycupKdPQHUZ
@fornewid
fornewid merged commit faa2374 into develop Oct 4, 2026
1 check passed
@fornewid
fornewid deleted the remove-redundant-declarations branch October 4, 2026 03:58
fornewid added a commit that referenced this pull request Oct 4, 2026
Removing the explicit kotlin-stdlib-jdk8 declarations in #346 let
okhttp 4.12.0 pull kotlin-stdlib-jdk7/jdk8 down to 1.8.21. Add
kotlin-bom, at the Kotlin plugin's version, as a platform in
configureKotlin(), so the Android modules resolve Kotlin artifacts at
2.2.21 again.


Claude-Session: https://claude.ai/code/session_01Upxj1FvQeDpycupKdPQHUZ

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
fornewid added a commit that referenced this pull request Oct 4, 2026
#346 moved app's /release rule to the root .gitignore as /app/release/
and deleted app/.gitignore. That went beyond removing the module
.gitignore files that only contained /build, so restore app/.gitignore
with /release and drop the root rule.


Claude-Session: https://claude.ai/code/session_01Upxj1FvQeDpycupKdPQHUZ

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant