Skip to content

[Gradle] Delete KotlinSourceSet.implementationMetadataConfigurationName - #7605

Open
Egor Andreevich (Egorand) wants to merge 1 commit into
masterfrom
egorand/260820/kt-80897-remove-implementation-metadata-configuration-name
Open

[Gradle] Delete KotlinSourceSet.implementationMetadataConfigurationName#7605
Egor Andreevich (Egorand) wants to merge 1 commit into
masterfrom
egorand/260820/kt-80897-remove-implementation-metadata-configuration-name

Conversation

@Egorand

Copy link
Copy Markdown
Contributor

^KT-80897 Verification Pending

@kotlin-safemerge

kotlin-safemerge Bot commented Aug 20, 2026

Copy link
Copy Markdown

Code Owners

RuleOwnersApproval
/​libraries/​tools/​kotlin-​gradle-​plugin-​api/​, /​libraries/​tools/​kotlin-​gradle-​plugin/​
kotlin-build-tools

timofeys1 🔒
PR commands for maintainers
CommandDescriptionParameters
/safe-mergeRebase-merges with automatic fixup commit squashing--fixup Autosquash fixup commits (on by default)
/safe-squash-mergeSquash-merges with optional commit title/body override--title Title of the squashed commit
--message Body of the squashed commit
/dry-runRuns the test pipeline with changes rebased on latest master--retry Retry the CI run on failure
/test-publicTriggers the public test suite without rebasing on latest master
/test-privateTriggers the private test suite without rebasing on latest master
/codeownersTriggers code owners check and comment update
/fixupSquashes fixup commits and force pushes the branch
/cancel-coordinatorCancels the merge coordinator currently running for this branch

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.

/final

/**
* @suppress
*/
@Deprecated(message = "KT-80897. Keep ABI compatibility with kotlinx-benchmarks", level = DeprecationLevel.HIDDEN)

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.

kotlinx-benchmark does not reference this API anymore (commit that removed the callsite) since 0.4.15 (latest version is 0.4.17).

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 think you might still not be able to remove it because coroutines seems to still use 0.4.13

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.

If that's the case, you will need to bump the version in coroutines and then whether we can remove this depend on where exactly the benchmarks were used

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.

Hmm, yeah, we had a similar issue with kotlinx.collections.immutable: https://jetbrains.slack.com/archives/C06QB5UHN9Y/p1787063796944369?thread_ts=1787058758.326779&cid=C06QB5UHN9Y. Let me try to bump the version in coroutines and see if it builds with the latest benchmarks.

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.

Stopped safe-merge job for now while I investigate.

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.

@Egorand

Copy link
Copy Markdown
Contributor Author

/safe-merge

@KotlinBuild

This comment has been minimized.

@Egorand

Copy link
Copy Markdown
Contributor Author

/safe-merge

@KotlinBuild

Build Server (KotlinBuild) commented Aug 20, 2026

Copy link
Copy Markdown

Quality gate is triggered at https://buildserver.labs.intellij.net/build/1036959519 — use this link to get full insight.

Quality gate was triggered with the following revisions:

kotlin
Branch: refs/merge/GITHUB-7605/safe-merge
Commit: 628fcb0


Triggered a retry attempt №1 out of 1.


Quality gate failed. See https://buildserver.labs.intellij.net/build/1036959519 to get full insight.

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.

3 participants