Skip to content

Update dependencies & migrate to AGP9#29

Merged
Syntey merged 7 commits into
mainfrom
feature/update-dependencies
Mar 23, 2026
Merged

Update dependencies & migrate to AGP9#29
Syntey merged 7 commits into
mainfrom
feature/update-dependencies

Conversation

@Syntey
Copy link
Copy Markdown
Contributor

@Syntey Syntey commented Mar 20, 2026

  • AGP9
  • Arkitekt 6
  • TargetSdk set to 36
  • Added CLAUDE.md

Comment thread app/build.gradle.kts Outdated
Comment on lines +51 to +59
named("main") {
kotlin.directories += setOf("src/main/kotlin")
}
named("test") {
kotlin.directories += setOf("src/test/kotlin")
}
named("androidTest") {
kotlin.directories += setOf("src/androidTest/kotlin")
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this still needed?

Comment thread CLAUDE.md
Copy link
Copy Markdown
Member

@matejsemancik matejsemancik Mar 23, 2026

Choose a reason for hiding this comment

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

We might need init script to remove / edit this after cloning the template

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Why?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The architecture-related part is fine, but project and business-related part will get outdated as you init the project and start working on it. Maybe we could just delete template-related stuff from it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Cleaned it up a little bit 👍🏻

@Syntey Syntey merged commit a9aea02 into main Mar 23, 2026
1 check passed
@Syntey Syntey deleted the feature/update-dependencies branch March 23, 2026 12:52
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