Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Switch from findbugs to spotbugs? #1433

Closed
pocmo opened this issue Oct 4, 2017 · 5 comments
Closed

Switch from findbugs to spotbugs? #1433

pocmo opened this issue Oct 4, 2017 · 5 comments
Labels
build help wanted P5
Milestone
v4.2

Comments

@pocmo
Copy link
Contributor

pocmo commented Oct 4, 2017

Following mozilla-mobile/telemetry-android#41 - Should we switch from findbugs to spotbugs?

Spotbugs:
https://github.com/findbugsproject/findbugs#readme
@pocmo pocmo added the build label Oct 4, 2017
@liuche
Copy link
Contributor

liuche commented Nov 7, 2017

This involves a small change to our build config.

@lime124 lime124 modified the milestone: Reserve Backlog Nov 10, 2017
@grubdragon
Copy link
Contributor

Could I take this up?

@mcomella
Copy link
Contributor

mcomella commented Jan 8, 2018

@grubdragon Sure! The steps for this bug:

Thanks for your help! :)

@mcomella mcomella mentioned this issue Jan 8, 2018
Merged
@ghost ghost mentioned this issue Mar 15, 2018
Closed
@ghost
Copy link

ghost commented Mar 15, 2018

Why to switch to Spotbugs?

@grubdragon
Copy link
Contributor

Thanks @vladioan3, I'm short of time these days.

pocmo pushed a commit to pocmo/focus-android that referenced this issue Mar 20, 2018
@pocmo
Switch from findbugs to spotbugs mozilla-mobile#1433
10c052a
pocmo pushed a commit that referenced this issue Mar 20, 2018
@pocmo
Switch from findbugs to spotbugs #1433
1269cec
@pocmo pocmo modified the milestones: Reserve Backlog, v4.2 Mar 20, 2018
@pocmo pocmo closed this as completed Mar 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
build help wanted P5
Projects
None yet
Milestone
v4.2
Development

No branches or pull requests
5 participants
@pocmo @lime124 @mcomella @liuche @grubdragon