Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Remove `StringScriptSource`, to be replaced with the existing `TextResourceScriptSource`.

    • -5
    • +3
    ./kotlin/dsl/PluginDependenciesSpecScopeTest.kt
  1. … 10 more files in changeset.
Upgrade JUnit version (#12924)

Upgrade JUnit to 4.13, JUnit platform to 5.6.2

    • -1
    • +1
    ./kotlin/dsl/DelegatedGradlePropertiesExtensionsTest.kt
    • -1
    • +1
    ./kotlin/dsl/ExtraPropertiesExtensionsTest.kt
    • -1
    • +1
    ./kotlin/dsl/ObjectFactoryExtensionsTest.kt
    • -1
    • +1
    ./kotlin/dsl/caching/LoadDirectoryTest.kt
    • -1
    • +1
    ./kotlin/dsl/provider/ScriptApiTest.kt
    • -1
    • +1
    ./kotlin/dsl/resolver/CompactTreeTest.kt
  1. … 322 more files in changeset.
Since 6.5

Signed-off-by: CristianGM <cristiangm@gmail.com>

Signed-off-by: cristian-garcia <cristian.garcia@scmspain.com>

  1. … 2 more files in changeset.
Treat Kotlin warnings as errors in the codebase

    • -0
    • +1
    ./kotlin/dsl/BuildCacheConfigurationExtensionsTest.kt
  1. … 17 more files in changeset.
Tell JaCoCo to ignore kotlin-dsl generated files

As stated in the issue #10956 the code generated by Gradle affects severely to our coverage.

I've created the annotation at org.gralde.api.internal because I expect no one uses it, it can be named differently while it contains Generated in the name

I haven't added a test checking this explicitly, no idea where to do it properly, but checked on out projects and there is still a small issue, accessors that contain methods with default arguments are still shown as not covered.

Signed-off-by: CristianGM <cristiangm@gmail.com>

Signed-off-by: cristian-garcia <cristian.garcia@scmspain.com>

  1. … 3 more files in changeset.
Upgrade Kotlin to 1.3.72

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 35 more files in changeset.
Invalidate instant execution cache upon changes to Kotlin scripts

By making the Kotlin DSL interpreter emit `onScriptClassLoaded` events.

At this point we don't yet make a distinction between init scripts and other

types of scripts.

    • -3
    • +13
    ./kotlin/dsl/execution/InterpreterTest.kt
  1. … 10 more files in changeset.
Update Gradle API extensions generated jar hash

in reproducibility test

Signed-off-by: Paul Merlin <paul@gradle.com>

Fix Kotlin DSL code generation fixture

The configuration creation block was causing an ambiguous

method selection error.

    • -0
    • +1
    ./kotlin/dsl/ScriptHandlerScopeTest.kt
  1. … 4 more files in changeset.
Fix Kotlin DSL constraints block

This commit introduces a workaround for the Kotlin DSL

which doesn't delegate the constraints block to the right

delegate type. The consequence is that constraints were

mixed with dependencies when using extension methods defined

on the `DependencyConstraintHandlerScope`, because the compiler

didn't realize that the receiver type was indeed this scope.

    • -8
    • +5
    ./kotlin/dsl/DependencyHandlerExtensionsTest.kt
    • -6
    • +3
    ./kotlin/dsl/ScriptHandlerScopeTest.kt
  1. … 5 more files in changeset.
Adjust KotlinScriptCompilerTest to work with Kotlin 1.3.70

By moving the script template class to the top level.

Catch up with master

  1. … 415 more files in changeset.
Add offending file name to exception caused by empty jar in the classpath

  1. … 1 more file in changeset.
Prove the new script templates are not missing any members

- prove new init script template is not missing any members from the legacy

`InitScriptApi`

- prove new setting script template is not missing any members from the legacy

`SettingsScriptApi`

    • -6
    • +40
    ./kotlin/dsl/provider/ScriptApiTest.kt
Upgrade Kotlin to 1.3.60

  1. … 7 more files in changeset.
Fix `ClassNotFoundExeception` when loading objects whose class is defined in a build script from the instant execution cache, after recreating the cache from a daemon process that has previously successfully used the cache.

For example, running `gradle taskA`, `gradle taskA`, `gradle taskB`, `gradle taskB` would fail if `taskB` uses types from a build script.

This was happening because the script ClassLoaders are cached and reused, but the association between ClassLoader and scope was lost, and this association is what instant execution uses to know how to load the class.

This change fixes one case of this problem, but the same problem can still happen if the set of build scripts being referenced changes.

    • -4
    • +5
    ./kotlin/dsl/execution/InterpreterTest.kt
  1. … 25 more files in changeset.
Ensure `settings.gradle.kts` with `pluginManagement` block honors auto-applied plugins

Make `settings.gradle.kts` evaluation trigger auto-applied plugins

  1. … 1 more file in changeset.
Integrate with Gradle Enterprise plugin 3.0 (#11041)

    • -7
    • +7
    ./kotlin/dsl/PluginDependenciesSpecScopeTest.kt
  1. … 52 more files in changeset.
Make paths/names of included builds immutable (#10998)

Changes the “build path” for included builds to be determined at inclusion time based on the directory name, or a user supplied override. Previously, we tried to use the root project name defined in the included build. This caused a lot of complexity due to it not being known until part way through building the included build.

This change also disallows use of `buildSrc` as a project name, as it collides with the `buildSrc` nested build.

  1. … 72 more files in changeset.
Polish ResolverCoordinatorTest

Signed-off-by: Paul Merlin <paul@gradle.com>

Prefer `Action<T>` over `T.() -> Unit` for the Kotlin script API

From the perspective of an user of the API both types are effectively

the same.

From a compile time and runtime perspective `Action<T>` seems a better

choice as values can be passed to the core Gradle API directly.

From an API consistency point of view `Action<T>` also looks like the

superior choice as it is more consistent with the rest of the Gradle

API.

And finally, from an user education perspective, `KotlinScript`

provides a clear opportunity for users to understand the equivalence

between `Action<T>` and `T.() -> Unit`.

    • -3
    • +3
    ./kotlin/dsl/provider/ScriptApiTest.kt
  1. … 2 more files in changeset.
Polish `ScriptApiTest`

    • -5
    • +4
    ./kotlin/dsl/provider/ScriptApiTest.kt
Polish `ResidualProgramCompilerTest`

- Remove spurious empty line

Remove `ScriptApi` interface and use `KotlinScript` instead

    • -2
    • +4
    ./kotlin/dsl/provider/ScriptApiTest.kt
  1. … 1 more file in changeset.
Restore Kotlin DSL script deps resolver short circuit

Signed-off-by: Paul Merlin <paul@gradle.com>

  1. … 3 more files in changeset.
Polish `TestWithCompiler`

- Reuse `PluginManagementSpec` mock

    • -1
    • +1
    ./kotlin/dsl/execution/TestWithCompiler.kt
Polish `ParserToCompilerTest`

Let `pluginManagement` Kotlin script template have an implicit `Settings` receiver

    • -5
    • +12
    ./kotlin/dsl/execution/TestWithCompiler.kt
    • -5
    • +10
    ./kotlin/dsl/provider/ScriptApiTest.kt
  1. … 2 more files in changeset.
Merge branch 'release' into bamboo/kotlin-dsl/implicit-receivers

    • -11
    • +9
    ./kotlin/dsl/provider/ScriptApiTest.kt
  1. … 3 more files in changeset.