Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

#43 version name is unspecified #44

Merged
merged 7 commits into from
Apr 12, 2015
Merged

Conversation

ouchadam
Copy link
Contributor

Deprecates the "version" param is favour of "publishVersion" to avoid conflicts with gradle's project.version when using properties

Fixes unspecified version name by applying the same property finding logic to both the maven publication and the bintray configuration

@ouchadam
Copy link
Contributor Author

fixes #43

}

def getPublishVersion() {
getString(project, 'publishVersion', extension.publishVersion == null ? extension.version : extension.publishVersion)
Copy link
Contributor

Choose a reason for hiding this comment

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

getString(project, 'publishVersion', extension.publishVersion ?: extension.version)

http://docs.groovy-lang.org/latest/html/documentation/index.html#_elvis_operator

The "Elvis operator" is a shortening of the ternary operator. Basically means if not null null return myself

with the Elvis operator, the value which is tested is used if it is not false or null

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that is such a good idea 👍

@blundell
Copy link
Contributor

nice action 👍

@ouchadam
Copy link
Contributor Author

@blundell comments addressed

@blundell
Copy link
Contributor

👍

blundell added a commit that referenced this pull request Apr 12, 2015
@blundell blundell merged commit 449aa66 into master Apr 12, 2015
@blundell blundell deleted the 43_version_name_is_unspecified branch April 12, 2015 19:22
@xrigau
Copy link
Contributor

xrigau commented Apr 13, 2015

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants