-
Notifications
You must be signed in to change notification settings - Fork 102
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
Update dependencies and clean up build scripts #423
Open
FenstonSingel
wants to merge
6
commits into
master
Choose a base branch
from
kotlin-community/build-scripts-cleanup
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Commits on Aug 9, 2024
-
Rationale is a slightly better DX when working with a version catalog (which is to be introduced in a later commit). Usages of Property API now leverage the assignment Kotlin compiler plugin instead of invoking `set` methods directly. Usages of deprecated Gradle API are either replaced with supported alternatives or removed due to not really doing anything.
Configuration menu - View commit details
-
Copy full SHA for 90dcbfb - Browse repository at this point
Copy the full SHA 90dcbfbView commit details -
KGP now uses the `.kotlin` folder for purposes that it previously used the `.gradle` folder for; so the `.kotlin` folder is now excluded from the Git repository as well. Kotlin-related Gradle properties from `gradle.properties` are cleaned up and actualized: * `kotlin.js.compiler` is removed * `kotlin.native.ignoreIncorrectDependencies` is prepared for its deprecation in 2.0.20 * `kotlin.wasm.stability.nowarn` is temporarily added until its deprecation in 2.0.20 Usages of deprecated `kotlinOptions` API are migrated to supported `compilerOptions` API (see KT-57292 for more details). (Application of `-version` to Kotlin compilation tasks is just removed altogether; applying it only to K/JVM tasks seems questionable, and applying it to K/N tasks stops them from actually compiling anything.) Usages of all experimental KGP API now opt into said API. Usages of all outdated KGP API are removed. An error suppression warning (see KT-61129 for more details) is disabled due to a high number of hits throughout the repository. A usage of deprecated `kotlin.native.ThreadLocal` annotation is migrated to supported `kotlin.native.concurrent.ThreadLocal` annotation. Workarounds for the following issues are removed from the repository's build scripts: * node.js not supporting "the latest Wasm" * node.js not supporting Wasm GC milestone 4 * KT-58303
Configuration menu - View commit details
-
Copy full SHA for fd315d9 - Browse repository at this point
Copy the full SHA fd315d9View commit details -
Remove old infrastructure for builds as a Kotlin user project
It will be reintroduced in a later commit.
Configuration menu - View commit details
-
Copy full SHA for e2c0d4e - Browse repository at this point
Copy the full SHA e2c0d4eView commit details -
Clean up and organize the repository's build scripts
There are three potentially important changes in this commit: * a workaround for Bintray publishing is removed because of, to quote a gradle.git#11412 commentator, "JFrog [electing] to shut down JCenter and Bintray rather than fix the problem" * `:kotlinx-datetime:compileCommonJsMainKotlinMetadata` task is enabled back to preserve compilation correctness (see KT-66382 for more details) * `:benchmarks` module's `jmh` source set is now also compiled during repository-wide builds (so that it's covered during builds as a Kotlin user project) Other than that, this commit is a bundle of minor, mostly structural tweaks for the sake of easier understanding and maintainability.
Configuration menu - View commit details
-
Copy full SHA for 90aa6b6 - Browse repository at this point
Copy the full SHA 90aa6b6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5898535 - Browse repository at this point
Copy the full SHA 5898535View commit details -
Configuration menu - View commit details
-
Copy full SHA for 94fce17 - Browse repository at this point
Copy the full SHA 94fce17View commit details
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.