Skip to content

Move remaining kotlin-analysis-api and gradle-plugin inline versions to the version catalog - #3174

Merged
hfmehmed merged 1 commit into
google:mainfrom
KlyneChrysler:build/2968-catalog-09-final-inline
Sep 3, 2026
Merged

Move remaining kotlin-analysis-api and gradle-plugin inline versions to the version catalog#3174
hfmehmed merged 1 commit into
google:mainfrom
KlyneChrysler:build/2968-catalog-09-final-inline

Conversation

@KlyneChrysler

Copy link
Copy Markdown
Contributor

Following #3127. This moves the last inline dependency versions in the production modules to the catalog, so no version literal remains in any non-fixture build file.

kotlin-analysis-api: annotations, lz4-java, opentelemetry-api. gradle-plugin: the lint-gradle lintChecks dependency.

Unlike the previous slice, all four are in use, so they are cataloged rather than removed: annotations is used directly in source, and lz4 and opentelemetry are isTransitive false runtime pins for the IntelliJ platform (opentelemetry is also relocated into the embeddable jar), so dropping them would risk a runtime NoClassDefFoundError.

Verified resolution is unchanged: :kotlin-analysis-api:dependencies for compile and runtime and :gradle-plugin:dependencies for lintChecks are identical before and after, 157 lines compared. Both modules compile.

The only versions still declared inline live in benchmark/exhaustive-processor, which I left alone as a benchmark fixture, consistent with excluding the examples and test resource projects. Happy to migrate it too if you would like it in scope for this issue.

…to the version catalog

Final inline versions for google#2968: annotations, lz4-java, and
opentelemetry-api in kotlin-analysis-api, and the lint-gradle lintChecks
dependency in gradle-plugin. Unlike the previous slice these are all in
use, annotations directly in source and lz4 and opentelemetry as
isTransitive false runtime pins for the IntelliJ platform, so they are
cataloged rather than removed.

@jaschdoc jaschdoc left a comment

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.

Thanks, looks good! @hfmehmed please take a look as well :)

@hfmehmed
hfmehmed added this pull request to the merge queue Sep 3, 2026
Merged via the queue into google:main with commit 7a11740 Sep 3, 2026
28 checks passed
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