chore: tidy up and harden .gitignore - #37
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Consolidates and expands
.gitignore, which had accumulated duplicate entries (*.logtwice, the Android Studio block twice,captures/three times) and was missing rules for real build output that was leaking intogit status.The immediate trigger:
composeApp/release/baselineProfiles/**/*.dmshowed up as untracked. The existing rule was.dm, which matches a file named.dm— not the extension — so signed-APK wizard output was never ignored.Changes
/*/release/,*.dm,*.aab. Scoped to module roots (/*/release/) deliberately, so it can never shadow an Android variant source set such ascomposeApp/src/release/..vscode/,google-services.json,*.hprof,*.tmp,*.swp,*.iws,*.pid,test-results/,test-reports/,.kotlin_cache/,kotlinc/,kotlin-tooling-box/,*.xcarchive,/gradle/gradle-daemon-jvm.properties..claude/settings.local.json— previously only ignored via a contributor's global git ignore, so a fresh clone would have surfaced (and could have committed) machine-local settings.*.xcodeproj/*.xcworkspacerules were deliberately not added:project.pbxproj, the shared scheme, andcontents.xcworkspacedataare tracked, and a blanket rule would silently ignore any newly added file under the project.Test plan
git ls-files -i -c --exclude-standardreturns empty — no tracked file is shadowed by the new rules.git check-ignore -vconfirmscomposeApp/release/*.apkandcomposeApp/release/**/*.dmare ignored, while a hypotheticalcomposeApp/src/release/res/values/strings.xmlis not../gradlew :composeApp:assembleDebugwith onlyANDROID_HOMEset (nolocal.properties) — builds successfully, confirming nothing the build needs is missing from a fresh checkout.path =reference iniosApp.xcodeproj/project.pbxprojagainst the fresh clone; the only absent entries are theWorn.app/WornShareExtension.appexbuild products.local.properties, a keystore, orgoogle-services.json, so ignoring them changes nothing for contributors.Checklist
./gradlew detektpasses (n/a — no Kotlin sources changed; verified via the fresh-clone build)🤖 Generated with Claude Code