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

Install spotbugs packages before testing gradle plugin #369

Merged
merged 1 commit into from
Sep 11, 2017

Conversation

KengoTODA
Copy link
Member

Refer discussion at: f7a08d9

I confirmed that ./gradlew build can pass even after rm -rf ~/.m2/repository/com/github/spotbugs in local.

@KengoTODA KengoTODA self-assigned this Sep 8, 2017
@KengoTODA KengoTODA requested a review from ThrawnCA September 8, 2017 03:08
KengoTODA referenced this pull request Sep 8, 2017
* add test to find problem like #320

* close #320: iterate ClassesDirs to find all target files

* install not only spotbugs but also others

* move timing to install

* use both of central and local as repository

* update CHANGELOG

* Revert "move timing to install"

This reverts commit e997efb.

* refs #337: extract shared logic to Before method

* refs #337: add more tests

* shorten test by deleting temporal File object in field

* generate plugin classpath by gradle

this change lets us stop invoking `install` task before the `test`.

* Revert "generate plugin classpath by gradle"

This reverts commit 9952aa6.
Copy link
Contributor

@ThrawnCA ThrawnCA left a comment

Choose a reason for hiding this comment

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

Looks like a good solution.

@KengoTODA KengoTODA merged commit ee291a8 into spotbugs:master Sep 11, 2017
@KengoTODA KengoTODA deleted the add-task-dependency branch September 11, 2017 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants