Skip to content

Conversation

@grechkovlad
Copy link
Contributor

… regular, single/multiple-field value classes"

This partially reverts commit 01f1d61, kepping added tests.

Such renaming has to be synchronized with proper changes in debugger, otherwise it introduces severe regression and brings no value.

^KT-82138: Fixed
^KT-78413: Fixed
^KT-73995: Open

@grechkovlad
Copy link
Contributor Author

Context:
The changes were introduced in https://kotlin.jetbrains.space/p/kotlin/reviews/1466/timeline and, as predicted (https://kotlin.jetbrains.space/im/review/17CiJR1QL9zc?message=Nt4000Nt4&channel=3QkIrK450NYi) broke the debugger, yet the tests did not detect it 😞

… regular, single/multiple-field value classes"

This partially reverts commit 01f1d61, kepping added tests.

Such renaming has to be synchronized with proper changes in debugger,
otherwise it introduces severe regression and brings no value.

^KT-82138: Fixed
^KT-78413: Fixed
^KT-73995: Open
^KT-80227: Open
@grechkovlad grechkovlad force-pushed the rr/vgrechko/param-naming branch from a62634c to e2051e2 Compare November 10, 2025 11:06
@grechkovlad
Copy link
Contributor Author

/test-public

@JetBrains JetBrains deleted a comment from kotlin-safe-merge bot Nov 11, 2025
@JetBrains JetBrains deleted a comment from kotlin-safe-merge bot Nov 11, 2025
@grechkovlad
Copy link
Contributor Author

/safe-merge

@KotlinBuild
Copy link

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

Corresponding Merge-Request in ultimate is found.

Quality gate was triggered with the following revisions:

kotlin
Branch: refs/merge/GITHUB-5549/safe-merge
Commit: c709784

ultimate
Branch: refs/coordinator/safe-merge/refs/merge/GITHUB-5549/safe-merge
Commit: 6711565

@KotlinBuild
Copy link

Quality gate finished successfully.

@KotlinBuild KotlinBuild merged commit 05024cd into master Nov 11, 2025
1 check passed
@KotlinBuild KotlinBuild deleted the rr/vgrechko/param-naming branch November 11, 2025 12:48
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.

6 participants