Release notes for v0.7.1 claims that type aliases were added for `Instant` and `Clock` but they don't exist

[chaoren]

https://github.com/Kotlin/kotlinx-datetime/releases/tag/v0.7.1 says

• Add kotlinx.datetime.Instant and kotlinx.datetime.Clock type aliases to, respectively, kotlin.time.Instant and kotlin.time.Clock to make migration more convenient.

However, I couldn't find any type aliases in the code.

[dkhalanskyjb]

Here they are:

• https://github.com/Kotlin/kotlinx-datetime/blob/f829958a1bcbe7b24a11a91519cf57aaf3daa74b/core/common/src/DeprecatedInstant.kt
• https://github.com/Kotlin/kotlinx-datetime/blob/f829958a1bcbe7b24a11a91519cf57aaf3daa74b/core/common/src/DeprecatedClock.kt

[chaoren]

I'm not seeing them on the master branch:

• https://github.com/Kotlin/kotlinx-datetime/blob/master/core/common/src/DeprecatedInstant.kt
• https://github.com/Kotlin/kotlinx-datetime/blob/master/core/common/src/DeprecatedClock.kt

Not seeing them in the commit with the v0.7.1 tag either:

• https://github.com/Kotlin/kotlinx-datetime/tree/v0.7.1/core/common/src/DeprecatedInstant.kt
• https://github.com/Kotlin/kotlinx-datetime/tree/v0.7.1/core/common/src/DeprecatedClock.kt

[dkhalanskyjb]

That's true. master and the tagged releases contain the compatibility artifact for now. Since in the binary, the compatibility artifact is a superset of the normal release, it's more all-encompassing. When publishing a release, we apply #540 in a separate branch. Please see https://github.com/Kotlin/kotlinx-datetime/pull/562/files.

[chaoren]

Thanks for the explanation. I think it would be a good idea to have separate tags for v0.7.1 and v0.7.1-compat releases and corresponding sections in the change log. Currently, the v0.7.1 tag has the code for v0.7.1-compat but the release notes for v0.7.1. This mismatch is quite confusing.

[sergeych]

Please tell me what to do. I have literally hundreds of projects based on serializable kotlinx.time.Instant, while "new" kotlin.time.Instant seems not to be. Why in the hell they can't make it serializable in a compatible manner and what to di with existing code? migration cost is as high as some of my clients would probably ditch kotlin as a server and web client language....

[fzhinkin]

@sergeych, kotlin.time.Instant is Serializable, and the kotlinx-serialization support for it comes with 1.9.0 (see https://github.com/Kotlin/kotlinx.serialization/releases/tag/v1.9.0). Do you experience any serialization issues with the latest kx-serialization?

[dkhalanskyjb]

Replaced the v0.7.* tags, added v0.7.*-0.6.x-compat tags.