Skip to content

Migrated to Korro 0.2.2#1846

Open
AndreiKingsley wants to merge 8 commits into
masterfrom
update_korro
Open

Migrated to Korro 0.2.2#1846
AndreiKingsley wants to merge 8 commits into
masterfrom
update_korro

Conversation

@AndreiKingsley
Copy link
Copy Markdown
Collaborator

@AndreiKingsley AndreiKingsley commented May 6, 2026

Closes #1814.

@AndreiKingsley AndreiKingsley changed the title Migrated to Korro 0.2.1 but with bugs Migrated to Korro 0.2.2 May 13, 2026
@AndreiKingsley AndreiKingsley marked this pull request as ready for review May 13, 2026 18:58
Comment thread build.gradle.kts
subprojects {
this.version = rootProject.version

tasks.matching {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we should not use subprojects {} anymore. Instead, this logic should be in the relevant build-logic convention plugins

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

or only in the build files where korro is applied

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I mean, eventually we will probably have a korro convention plugin #1661 where this logic could live

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@Jolanrensen we will use korro only in :samples after migration (#898), so there's no need in korro convention plugin.

Can we just keep it for now?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hmm. I see you did write mustRunAfter logic in :core but not in :samples, which looks a bit odd. It's okay to do it without convention plugin, but then the logic should be in :samples, not in the root project

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.

Migrate korro to 0.2.1

2 participants