Skip to content
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

Avoid using command-line modification to enhance build execution #123

Merged
merged 6 commits into from
Nov 28, 2021

Conversation

bigdaz
Copy link
Member

@bigdaz bigdaz commented Nov 28, 2021

Instead of using command-line switches to control the way Gradle builds will execute, we now add control files to Gradle User Home. This reduces the coupling between gradle-build-action and the actual Gradle invocation, making it easier to use the action functionality without the action needing to "own" the Gradle invocation.

Instead of passing `--no-daemon` on the command line, the same
functionality is now acheived by writing a gradle.properties file
when initializing Gradle User Home.
Instead of writing this file to a temp directory and referencing it
on the Gradle command line, the init script is now written to Gradle
User Home so that it is picked up automatically.
The generated gradle.properties file and init script are not cached,
so need to be generated even when restoring from cache.
Instead of writing the URL to a file on disk, reading it later and
using the Actions API to record the output parameter and write the notice,
these things are now done directly via Actions commands emitted directly
from the init script.
@bigdaz bigdaz merged commit 92a1f98 into main Nov 28, 2021
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.

1 participant