Skip to content

Fix ability to specify flank version.#97

Merged
runningcode merged 1 commit into
masterfrom
no/property
Apr 24, 2020
Merged

Fix ability to specify flank version.#97
runningcode merged 1 commit into
masterfrom
no/property

Conversation

@runningcode

Copy link
Copy Markdown
Owner

We need to specify the version inside the afterEvaluate block
otherwise the extension isn't set yet.

This also uses the new property api.

This avoids touching the filesystem during the configuration phase of
Gradle.

More information on working with files can be found here:
https://docs.gradle.org/current/userguide/working_with_files.html

Comment thread README.md
``` groovy
fladle {
serviceAccountCredentials("${project.file("flank-gradle-service-account.json")}")
serviceAccountCredentials = project.layout.projectDirectory.file("flank-gradle-service-account.json")

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

breaking api change.

Comment thread README.md

### flankVersion
`flankVersion("flank_snapshot")` to specify a Flank snapshot.
`flankVersion = "flank_snapshot"` to specify a Flank snapshot.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

breaking api change

We need to specify the version inside the `afterEvaluate` block
otherwise the extension isn't set yet.

This also uses the new property api.

This avoids touching the filesystem during the configuration phase of
Gradle.

More information on working with files can be found here:
https://docs.gradle.org/current/userguide/working_with_files.html
@runningcode
runningcode merged commit 468b100 into master Apr 24, 2020
@runningcode
runningcode deleted the no/property branch April 24, 2020 22:55
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