Conversation
…itHub runs out of memory
aSemy
added a commit
to antohaby/dokkatoo
that referenced
this pull request
Aug 31, 2023
…ub Actions doesn't like them (OOM), so I'll deal with them in adamko-dev#115
aSemy
changed the title
aSemy
added a commit
that referenced
this pull request
Aug 31, 2023
…dle Plugin API (#114) * Use `compileDependencyFiles` as main source of compilation dependencies Additionally, for Native compilations explicitly include stdlib and platform dependencies from Kotlin Native distribution. Home location of Kotlin Native distribution is collected via Internal API of Kotlin Gradle Plugin. It is safe for few reasons: * konanHome property is accessed only for past Kotlin versions (up to 1.9.X) * In Kotlin 2.0 `compileDependencyFiles` will include native-specific dependencies * Don't add suppressed source sets to generator task Inputs This prevents from unnecessary dependency resolutions, including compile tasks. Particularly test source sets depend on compilations output of main source sets. And if test source sets are not skipped then Kotlin compilation will be triggered unnecessary. * Bump kotlin version in multiplatform-example project Kotlin 1.9.0 supports Gradle Configuration Cache for Multiplatform projects Dokka generation task now depends on `transform{sourceSetName}DependenciesMetadata` tasks from Kotlin Gradle Plugin that didn't support Gradle Configuration Cache in earlier versions of Kotlin. Making whole gradle build incompatible with configuration cache. * remove unused 'compileDependencyFiles' * tidy up GradleTestKitUtils * test more Kotlin versions and add K/N targets in KMP functional tests * disable Gradle Daemon in Gradle TestKit tests * fix Kotlin Native tests in KotlinMultiplatformFunctionalTest * refactoring & tidying - use KotlinToolingVersion to get the Kotlin version - rename 'isMain' properties/functions to 'isPublished', because it makes a bit more sense - fetch the konanHome prop by using extraProperties * KotlinMultiplatformFunctionalTest moved to a separate PR because GitHub Actions doesn't like them (OOM), so I'll deal with them in #115 --------- Co-authored-by: Adam <897017+aSemy@users.noreply.github.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
additional KMP function tests for #114, but split out into a separate PR because GitHub runs out of memory