Compatibility guide for Kotlin 2.1.0 #4540
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tapchicoma
reviewed
Nov 11, 2024
Tapchicoma
approved these changes
Nov 12, 2024
zarechenskiy
requested changes
Nov 19, 2024
zarechenskiy
approved these changes
Nov 21, 2024
ilya-g
requested changes
Nov 22, 2024
Comment on lines
134
to
135
| > - 2.1.0: report a warning | ||
| > - 2.2.0: raise the warning to an error |
There was a problem hiding this comment.
Suggested change
| > - 2.1.0: report a warning | |
| > - 2.2.0: raise the warning to an error | |
| > - 2.1.0: start throwing an exception when a map structural modification is detected |
There was a problem hiding this comment.
thank you - I updated it (removed the start from the beginning)
- 2.1.0: throw an exception when a map structural modification is detected
qurbonzoda
reviewed
Nov 22, 2024
qurbonzoda
reviewed
Nov 25, 2024
AlejandraPedroza
requested changes
Nov 25, 2024
| [//]: # (title: Compatibility guide for Kotlin 2.1) | ||
|
|
||
| _[Keeping the Language Modern](kotlin-evolution-principles.md)_ and _[Comfortable Updates](kotlin-evolution-principles.md)_ are among the fundamental principles in | ||
| Kotlin Language Design. The former says that constructs which obstruct language evolution should be removed, and the |
There was a problem hiding this comment.
I see this part is fixed from previous guide versions, but, to make it more conversational and closer to users, could we change "The former and the latter", for:
"The first principle states that constructs which obstruct language evolution should be removed, while the second principle emphasizes that such removal should be well-communicated..."
or something like this?
|
|
||
| In this document we introduce several kinds of compatibility: | ||
|
|
||
| - _source_: source-incompatible change stops code that used to compile fine (without errors or warnings) from compiling |
There was a problem hiding this comment.
Should we bold the terms? For consistency with this type of list in our docs.
AlejandraPedroza
approved these changes
Nov 25, 2024
qurbonzoda
approved these changes
Nov 25, 2024
ilya-g
reviewed
Nov 25, 2024
| @@ -0,0 +1,479 @@ | |||
| [//]: # (title: Compatibility guide for Kotlin 2.1) | |||
There was a problem hiding this comment.
A related question: would it be possible to link the compatibility guide from the What's new page? (as we did in 1.9 and before)
There was a problem hiding this comment.
Hi Ilya - I asked the team - we'll link to it! 👍
ilya-g
approved these changes
Nov 25, 2024
sarahhaggarty
approved these changes
Nov 25, 2024
| > | ||
| > - 1.4.30: introduce locale-agnostic alternatives as experimental API | ||
| > - 1.5.0: deprecate locale-sensitive case conversion functions with a warning | ||
| > - 2.1.0: raise the warning to an error` |
There was a problem hiding this comment.
Suggested change
| > - 2.1.0: raise the warning to an error` | |
| > - 2.1.0: raise the warning to an error |
| > **Incompatible change type**: source | ||
| > | ||
| > **Short summary**: Freezing-related APIs in Kotlin/Native, previously marked with the `@FreezingIsDeprecated` annotation, are now deprecated. | ||
| > This aligns with the new memory manager that removes the need for freezing objects for thread sharing. |
There was a problem hiding this comment.
Suggested change
| > This aligns with the new memory manager that removes the need for freezing objects for thread sharing. | |
| > This aligns with the introduction of the new memory manager that removes the need for freezing objects for thread sharing. |
| > | ||
| > **Incompatible change type**: behavioral | ||
| > | ||
| > **Short summary**: K2 changes the behavior of smart cast propagation by introducing bidirectional propagation of type |
There was a problem hiding this comment.
Suggested change
| > **Short summary**: K2 changes the behavior of smart cast propagation by introducing bidirectional propagation of type | |
| > **Short summary**: The K2 compiler changes its behavior for smart cast propagation by introducing bidirectional propagation of type |
| > | ||
| > - 1.4.30: introduce locale-agnostic alternatives as experimental API | ||
| > - 1.5.0: deprecate locale-sensitive case conversion functions with a warning | ||
| > - 2.1.0: raise the warning to an error` |
There was a problem hiding this comment.
Suggested change
| > - 2.1.0: raise the warning to an error` | |
| > - 2.1.0: raise the warning to an error |
| > | ||
| > - 2.1.0: deprecate and remove `kotlin-stdlib-common.jar` artifact | ||
|
|
||
| ### Deprecate appendln in favor of appendLine |
There was a problem hiding this comment.
Suggested change
| ### Deprecate appendln in favor of appendLine | |
| ### Deprecate appendln() in favor of appendLine() |
| > | ||
| > **Short summary**: Exposing types with lower visibility through type parameter bounds is now prohibited, | ||
| > addressing inconsistencies in type visibility rules. | ||
| > This change ensures that bounds on type parameters follow the same visibility rules as classes, preventing issues like IR validation errors in JVM. |
There was a problem hiding this comment.
Suggested change
| > This change ensures that bounds on type parameters follow the same visibility rules as classes, preventing issues like IR validation errors in JVM. | |
| > This change ensures that bounds on type parameters follow the same visibility rules as classes, preventing issues like IR validation errors in the JVM. |
| > | ||
| > **Short summary**: The `registerKotlinJvmCompileTask(taskName, moduleName)` function is deprecated | ||
| > in favor of the new `registerKotlinJvmCompileTask(taskName, compilerOptions, explicitApiMode)` function, which now accepts `KotlinJvmCompilerOptions`. | ||
| > This allows you to pass a `compilerOptions` instance, typically from an extension or target, with values used as conventions for the task’s options. |
There was a problem hiding this comment.
Suggested change
| > This allows you to pass a `compilerOptions` instance, typically from an extension or target, with values used as conventions for the task’s options. | |
| > This allows you to pass a `compilerOptions` instance, typically from an extension or target, with values used as conventions for the task's options. |
| > | ||
| > **Incompatible change type**: source | ||
| > | ||
| > **Short summary**: support for these platform plugin IDs have been removed: |
There was a problem hiding this comment.
Suggested change
| > **Short summary**: support for these platform plugin IDs have been removed: | |
| > **Short summary**: Support for these platform plugin IDs have been removed: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Compatibility guide for Kotlin 2.1