Skip to content

Move kotlin-analysis-api IntelliJ platform dependencies to the version catalog - #3105

Open
KlyneChrysler wants to merge 2 commits into
google:mainfrom
KlyneChrysler:build/2968-catalog-07-kaa-intellij-deps
Open

Move kotlin-analysis-api IntelliJ platform dependencies to the version catalog#3105
KlyneChrysler wants to merge 2 commits into
google:mainfrom
KlyneChrysler:build/2968-catalog-07-kaa-intellij-deps

Conversation

@KlyneChrysler

Copy link
Copy Markdown
Contributor

First slice of the inline build file versions remaining for #2968, continuing from #3102.

Five org.jetbrains.intellij.deps artifacts move from hardcoded versions to catalog accessors: jna, jna-platform, trove4j, log4j in the originalLog4j configuration, and jdom. The two jna artifacts share one version entry, and the jnaPlatform alias uses the camelCase leaf so no asProvider is needed, per the earlier reviews.

Verified that resolution is unchanged: ./gradlew :kotlin-analysis-api:dependencies output for the compile, runtime, and originalLog4j configurations is identical before and after this commit, 165 lines compared. :kotlin-analysis-api:compileKotlin passes.

…n catalog

First slice of the inline versions remaining for google#2968: jna,
jna-platform, trove4j, log4j, and jdom, all from
org.jetbrains.intellij.deps. The jna artifacts share one version, and
jnaPlatform uses the camelCase leaf so no asProvider is needed.

@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 again! As per usual, I will let @hfmehmed take a look as well.

@jaschdoc

jaschdoc commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

@KlyneChrysler, if you can avoid rebasing or merging main unless there are merge conflicts, that would be great. In the future, it makes it easier and faster for us to merge your PR :)

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