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

Allow projects without a SourceSet (Eg. Android) to use this SpotBugs… #150

Closed
wants to merge 1 commit into from

Conversation

abishek3876
Copy link

…Task

If a project does not contain a sourceset, the execution of type SpotBugsTask fails with path may not be null or empty string. path='null' error.
This is because the sourceset is passed as an @InputFiles and it is null.

…Task

If a project does not contain a sourceset, the execution of type SpotBugsTask fails with `path may not be null or empty string. path='null'` error.
This is because the sourceset is passed as an `@InputFiles` and it is null.
Copy link
Member

@KengoTODA KengoTODA left a comment

Choose a reason for hiding this comment

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

Thanks for your PR! Could you consider to:

  1. Update CHANGELOG.md based on Keep a Changelog 1.0.0
  2. Make an unit test to reproduce your problem, then we can make sure that this feature won't be broken in future.

@abishek3876
Copy link
Author

Sure... Will try to add them up over the weekend.

@rompic
Copy link

rompic commented Aug 29, 2019

is there any update for this one? thanks!

@KengoTODA
Copy link
Member

version 4 should work even for Android project with plugin version 3.5, please have a try.

but it seems that Android plugin 3.6 removed task so this plugin doesn't work: #194
if you have idea to support both 3.5 and 3.6, please let us know.

@davidburstrom
Copy link

The Android Gradle Plugin version is available as a static field in the JAR. This is how the Android Pitest plugin does it: https://github.com/koral--/gradle-pitest-plugin/blob/master/src/main/groovy/pl/droidsonroids/gradle/pitest/PitestPlugin.groovy#L37

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.

4 participants