Skip to content

Remove deprecated APIs usage and enable warning-as-errors for build scripts #2925

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

Merged
merged 10 commits into from
Feb 21, 2025

Conversation

Tapchicoma
Copy link
Contributor

No description provided.

@sandwwraith
Copy link
Member

Also, PTAL on Kotlin master build failure. It seems we can't turn -Werror on builds while withJava is deprecated

@sandwwraith
Copy link
Member

You have to remove -Werror for now, wait for serialization to update to 2.1.20, remove withJava, and then this PR can be merged

@Tapchicoma Tapchicoma force-pushed the yahor/remove-deprecated-api-usage branch from 65b99c9 to 1a56460 Compare February 20, 2025 19:57
@Tapchicoma
Copy link
Contributor Author

I have to rebase to fix newly introduced warnings in build scripts (now first commit) plus suppress deprecation warning for withJava() DSL introduced in Kotlin 2.1.20. This suppression should be removed once project will be updated to Kotlin 2.1.20 by default.

@sandwwraith sandwwraith merged commit 5439653 into dev Feb 21, 2025
4 checks passed
woainikk pushed a commit that referenced this pull request Mar 5, 2025
…cripts (#2925)

* Fix deprecated URL constructor usage in build scripts.

* Enable warnings-as-errors for build scripts

This should force to update build scripts on any new deprecation introduced in the used plugins.

* Fix deprecation waring in teamcity-conventions.gradle.kts

* Suppress deprecation in native-targets-conventions.gradle.kts

* Use proper OptIn annotation in source-sets-conventions.gradle.kts

* Fix deprecation warnings in Java9Modularity.kt

* Suppress deprecation introduced in Kotlin 2.1.20 release.

Should be migrated once this project will start using Kotlin 2.1.20 by default.

* Enable warnings-as-errors for build logic compilation

This should force to update scripts on new deprecation introduced in the plugins.

* Enable warnings-as-errors for integration-tests project build scripts

* Remove deprecated 'kotlin.js.compiler' property

Only IR compiler available now.
Tapchicoma added a commit that referenced this pull request Mar 12, 2025
…cripts (#2925)

* Fix deprecated URL constructor usage in build scripts.

* Enable warnings-as-errors for build scripts

This should force to update build scripts on any new deprecation introduced in the used plugins.

* Fix deprecation waring in teamcity-conventions.gradle.kts

* Suppress deprecation in native-targets-conventions.gradle.kts

* Use proper OptIn annotation in source-sets-conventions.gradle.kts

* Fix deprecation warnings in Java9Modularity.kt

* Suppress deprecation introduced in Kotlin 2.1.20 release.

Should be migrated once this project will start using Kotlin 2.1.20 by default.

* Enable warnings-as-errors for build logic compilation

This should force to update scripts on new deprecation introduced in the plugins.

* Enable warnings-as-errors for integration-tests project build scripts

* Remove deprecated 'kotlin.js.compiler' property

Only IR compiler available now.
Tapchicoma added a commit that referenced this pull request Mar 12, 2025
woainikk pushed a commit that referenced this pull request Mar 21, 2025
* Remove deprecated APIs usage and enable warning-as-errors for build scripts (#2925)

* Fix deprecated URL constructor usage in build scripts.

* Enable warnings-as-errors for build scripts

This should force to update build scripts on any new deprecation introduced in the used plugins.

* Fix deprecation waring in teamcity-conventions.gradle.kts

* Suppress deprecation in native-targets-conventions.gradle.kts

* Use proper OptIn annotation in source-sets-conventions.gradle.kts

* Fix deprecation warnings in Java9Modularity.kt

* Suppress deprecation introduced in Kotlin 2.1.20 release.

Should be migrated once this project will start using Kotlin 2.1.20 by default.

* Enable warnings-as-errors for build logic compilation

This should force to update scripts on new deprecation introduced in the plugins.

* Enable warnings-as-errors for integration-tests project build scripts

* Remove deprecated 'kotlin.js.compiler' property

Only IR compiler available now.

* fixup! Remove deprecated APIs usage and enable warning-as-errors for build scripts (#2925)

* Drop usage of deprecated Any?.freeze() in K/N target (#2819)

The legacy memory model was removed from the K/N runtime in 1.9.20.
Now, this function does nothing, and its usages can be safely dropped.

Worker.execute functions no longer require the argument to be frozen,
as objects can no longer be frozen.

* Update KLib API dumps

* Update yarn lock

* Better approach to override language version

This allows propagating it to all sourcesets including shared one.

* Update to Gradle 8.7 in integration-test

* Update to Kotlin 2.1.0 in integration test

* Update yarn lock in integration-test

---------

Co-authored-by: Abduqodiri Qurbonzoda <abduqodiri.qurbonzoda@jetbrains.com>
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.

2 participants