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

Added Android Plugin support. Pulls in the Main Source. #17

Merged
merged 1 commit into from
Jun 25, 2014

Conversation

chrisjenx
Copy link
Contributor

This uses the Main sourceSet of the Android plugins (Android/Library) they both inherit from the BasePlugin now.

This shouldn't be nessary, but setting https://github.com/kt3k/coveralls-gradle-plugin/blob/master/src/main/groovy/org/kt3k/gradle/plugin/CoverallsPluginExtension.groovy#L26 has no effect on the project.extentions.coveralls.sourceSets property.

If you fix that then, this becomes slightly obsolete.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.76%) when pulling e2a1e22 on justyoyo:fix/android-plugin into 9253a4e on kt3k:master.

@kt3k
Copy link
Owner

kt3k commented Jun 25, 2014

Hi,

Thanks for your contribution!

Sorry for some inconvenience. project.extentions.coveralls.sourceSets is for now only in master branch and has not been released yet.

And even if we released that feature, this change would be useful for android projects because it enables users to upload android app coverage without configuration. (In my opinion, the capability without any configuration is important for this kind of tool.) So I'll merge this.

Thanks,

kt3k added a commit that referenced this pull request Jun 25, 2014
Added Android Plugin support. Pulls in the Main Source.
@kt3k kt3k merged commit 926f0e3 into kt3k:master Jun 25, 2014
@kt3k
Copy link
Owner

kt3k commented Jun 26, 2014

Deployed as v0.5.0.

Thanks,

@chrisjenx
Copy link
Contributor Author

@kt3k no worries.

@chrisjenx chrisjenx deleted the fix/android-plugin branch June 26, 2014 21:14
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