Remove legacy FindBugs tasks from build script #4040
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ezaquarii
commented
May 18, 2019
build.gradle
Outdated
| } | ||
|
|
||
| check.dependsOn 'checkstyle', 'findbugs', 'pmd', 'lint' | ||
| check.dependsOn 'checkstyle', 'pmd', 'lint' |
Collaborator
Author
There was a problem hiding this comment.
Not sure about this
Member
There was a problem hiding this comment.
This just means that ./gradlew check will start checkstyle, pmd, etc.
So having findbug/spotbug in it would be nice.
Collaborator
Author
There was a problem hiding this comment.
Ok, I added spotbugs here.
ezaquarii
force-pushed
the
ezaquarii/remove-findbugs
branch
from
8a78037 to
158251e
Compare
Codecov Report
@@ Coverage Diff @@
## master #4040 +/- ##
===========================================
+ Coverage 12.71% 14.3% +1.59%
Complexity 1 1
===========================================
Files 329 329
Lines 30998 30998
Branches 4424 4424
===========================================
+ Hits 3940 4435 +495
+ Misses 26394 25779 -615
- Partials 664 784 +120
|
Member
|
@ezaquarii a rebase should help with the ci build now (latest merge to master fixed the pipeline, fix provided by @tobiasKaminsky) |
Signed-off-by: Chris Narkiewicz <[email protected]>
ezaquarii
force-pushed
the
ezaquarii/remove-findbugs
branch
from
158251e to
d6b4dee
Compare
Collaborator
|
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/9387.apk |
Collaborator
Codacy297Lint
SpotBugs (new)
SpotBugs (master)
|
AndyScherzinger
approved these changes
May 23, 2019
tobiasKaminsky
added a commit
that referenced
this pull request
May 24, 2019
3b31841 Merge pull request #3969 from nextcloud/receivePushDelete 6074c93 Drone: update FindBugs results to reflect reduced error/warning count [skip ci] f28007e OCFileListFragment: Only scroll to top if directory changed (#4058) b061e4d Merge commit '51ee4d88fd0555a945a24b1ab3beba2ffdc79c06' 51ee4d8 handle silent delete/delete-all push notifications do not ignore talk messages (will only be sent if no talk app is installed) exclude generated Parcelable fb6b11f Use product name on rich document, if available (#3971) 4197d3b use proper qa build lib dependency 8c06097 Only compare directories if necessary 403a3e0 Use product name on rich document, if available use placeholder for string reset to master-snapshot fix analysis warning use new translatable string use var vor qa flavor too 9b0a789 Merge pull request #4040 from nextcloud/ezaquarii/remove-findbugs d6b4dee Remove legacy FindBugs tasks from build script 53c9c05 daily dev 20190523
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Chris Narkiewicz [email protected]