Commit Graph

9 Commits (describe-install)

Author SHA1 Message Date
Michael Comella 3e96f6b385 For #13959 - review: add clarifying comments. 4 years ago
Michael Comella 3b5d6d58d9 For #13959: add MozillaStrictModeSuppression detekt check. 4 years ago
Michael Comella 66f220c22a For #13959: improve custom detekt violation output.
Before it used to output the violations all one one line. Now it looks
like:
```
MozillaStrictModeSuppression:
    'import mozilla.components.support.ktx.android.os.resetAfter' at
(17,1) in /StrictModeManager.kt
    Please use `components.strictMode.resetAfter` instead because it has
performance improvements and additional code to monitor for performance
regressions.

MozillaStrictModeSuppression:
    'setThreadPolicy(threadPolicy.build())' at (56,24) in
/StrictModeManager.kt
    Please use `components.strictMode.resetAfter` instead because it has
performance improvements and additional code to monitor for performance
regressions.

MozillaStrictModeSuppression:
    'setVmPolicy(builder.build())' at (71,24) in /StrictModeManager.kt
    NOT YET IMPLEMENTED: please consult the perf team about
implementing`StrictModeManager.resetAfter`: we want to understand the
performance implications of suppressing setVmPolicy before allowing it.
```
4 years ago
Michael Comella f690f8736d No issue: replace LintUnitTestRunner with detekt implementation.
This should be more performant and easier to maintain.
4 years ago
Tiger Oakes 5a4c391b52 Suppress YamlConfig deprecation 4 years ago
Michael Comella f1159f3558 No issue: remove assertJ.
It was only used for 3 checks so it's not worth having a dependency on
it.
4 years ago
Emily Kager b94ac93ce9
Upgrade to detekt 1.6.0 (#9306)
Changelog:

https://arturbosch.github.io/detekt/changelog.html

Fixes #7360.  Also use more exact test glob.

Co-authored-by: Andrew Gaul <andrew@gaul.org>
4 years ago
Sawyer Blatz 20396f78a9
For #7698: Adds search back button animation (#7840)
* For #7698: Adds search back button animation

* Refactor for searchController
4 years ago
Will Hawkins f69009aa9e No Issue: Add custom detekt rule to blacklist certain properties
Add a custom detekt rule to blacklist certain properties. This is
immediately useful for making sure that developers do not configure
runtime behavior using the `BuildConfig.DEBUG` property but it is
useful in a wider context.
4 years ago