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
Update detekt to version 1.21.0 #2056
Conversation
0b04862
to
c12643c
Compare
More needs doing here still. |
e08f665
to
3ce5ff1
Compare
Alright, so the tl;dr of this PR:
Now I think it's ready for review 😄 |
3ce5ff1
to
4daff73
Compare
app/src/main/java/org/mozilla/reference/browser/addons/InstalledAddonDetailsActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mozilla/reference/browser/settings/AccountSettingsFragment.kt
Outdated
Show resolved
Hide resolved
Where is the upstream copy coming from? Did you manually apply it to our config file or was there some tool that generates the new values for us. There are a lot things which have been turned on, which I wonder if we can just ignore for now, or maybe it's not worth upgrading detekt? |
I created default configs for detekt 1.19 and 1.21 and diffed them to see what changes were made to the defaults by upstream. |
4daff73
to
fe78f34
Compare
ee7fb21
to
77fec26
Compare
77fec26
to
2e11949
Compare
Sitting on this until after mozilla-mobile/firefox-android#292 lands. Will update the PR to v1.22 and the same config options afterwards. |
Superseded by #2337 |
Turns out this upgrades without needing any fix-ups at all 🤘