Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[K2] List reference leads to the stdlib collections package #3375

Closed
atyrin opened this issue Nov 24, 2023 · 4 comments · Fixed by #3447
Closed

[K2] List reference leads to the stdlib collections package #3375

atyrin opened this issue Nov 24, 2023 · 4 comments · Fixed by #3447
Assignees
Labels
bug topic: K2 Issues / PRs that are related to the K2 migration. See #2888 upstream bug An issue that appears due to an upstream dependency bug, such as kotlin-compiler
Milestone

Comments

@atyrin
Copy link
Contributor

atyrin commented Nov 24, 2023

/**
 * ref to [List] or [Set]
 */
fun x(){}

List link leads to https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.collections/ instead of https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.collections/-list/ (in K1).
For Set the link is correct.

Installation

  • Dokka version: 1.9.20

Parent: #3328

@atyrin atyrin added bug topic: K2 Issues / PRs that are related to the K2 migration. See #2888 labels Nov 24, 2023
@atyrin
Copy link
Contributor Author

atyrin commented Nov 28, 2023

The same case I see for KMP project and String. Reproduces in common code(JVM+JS) and JS code. But ok for JVM

/**
 * [String]
 */
fun withString(s: String){}

The link from kdoc leads to https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.text/index.html. From signature to https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html

@vmishenev
Copy link
Member

@vmishenev vmishenev added the upstream bug An issue that appears due to an upstream dependency bug, such as kotlin-compiler label Dec 8, 2023
@IgnatBeresnev
Copy link
Member

Theoretically, we could add a workaround on our side (change the priority of resolved symbols), but ideally it needs to be fixed in analysis-api.

@vmishenev
Copy link
Member

The logic of ordering should be like in K1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug topic: K2 Issues / PRs that are related to the K2 migration. See #2888 upstream bug An issue that appears due to an upstream dependency bug, such as kotlin-compiler
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants