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

Feature/gradle kotlin dsl compatibility #216

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Feature/gradle kotlin dsl compatibility #216

wants to merge 2 commits into from

Conversation

teyckmans
Copy link

Hi,

I was converting my groovy gradle build files and was not able to assign publications on the extension class. These are really small changes that make it possible to use this plugin when using kotlin gradle build files.

What changed:

  • replaced all String[] attributes on the BintrayExtension by List and initialized them with empty lists.
  • configurations and publications can now be properly added by using .add(...) on the initialized list attributes of BintrayExtension.
  • aligned the types on the BintrayUploadTask for the licenses and labels attributes.
  • updated the empty check on the publications and configurations attributes in the BintrayPlugin class.

- replaced all String[] attributes on the BintrayExtension by List<String> and initialized them with empty lists.
- configurations and publications can now be properly added by using .add(...) on the initialized list attributes of BintrayExtension.
- aligned the attribute types on the BintrayUploadTask.
- update the empty check on the publications and configurations attributes in the BintrayPlugin class.
@GuilhE
Copy link

GuilhE commented Jun 8, 2020

Hello @teyckmans , do you know if this PR solves this issue.

@teyckmans
Copy link
Author

Hi @GuilhE , I don't think I had a problem accessing the extension it was just not that handy to use with the Kotlin build scripts.

In this build https://github.com/teyckmans/ui-rig/blob/master/ui-rig-core/build.gradle.kts I was testing the changes.

@CLAassistant
Copy link

CLAassistant commented Jun 18, 2020

CLA assistant check
All committers have signed the CLA.

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.

3 participants