Commit Graph

19 Commits

Author SHA1 Message Date
Michael Comella
94bbf90f3c No issue: move TestStrictModeManager to perf sub package. 2020-11-13 16:07:59 -08:00
Michael Comella
99600247bb No issue: move relevant files in :mozilla-lint-rules to perf pkg. 2020-11-13 16:07:59 -08:00
Michael Comella
c647679e4d No issue: move relevant :mozilla-detekt-rules to perf/ pkg. 2020-11-13 16:07:59 -08:00
Michael Comella
346f85868e No issue: perf code owner all /perf/ src dir.
Our previous strategy was to code owner all the specific files we should
own but that's fragile and unreasonable. This is simpler. We expect
there will still be edge cases that live outside these directories (e.g.
*Application*kt).
2020-11-13 16:07:59 -08:00
Michael Comella
72ab058551 For #16376: move Startup*Test to perf pkg under code owners.
Instead of listing many different files here, we should just put them in
/perf/ packages and code owner them.
2020-11-13 15:39:06 -08:00
MarcLeclair
7b1af41b40
15278 detekt rule runblocking (#15942)
* For #15278: added CoroutineManager to count runBlocking calls

* For #15278: Added actual detekt rule for runblocking and its config to the yaml

* For #15278: Added unit test for RunblockingCounter

* For #15278: renamed StrictModeStartupSuppressionCountTest.kt to PerformanceStartupTest.kt and added runBlockingCount test

* Lint fix

* For #15278: made runblocking a Long to prevent overflow

* For #15278: fixed MozRunblocking name, description and moved RunBlockingCounter to perf package

* For #15278:Renamed MozillaRunblockingCheck to MozillaRunBlockingCheck

* For #15278: Added setup for unit test, since it failed without restting counter

* For #15278: Fixed naming for RunBlocking lint check

* For #15278: removed changes made to test to use runBlockingIncrement

* For #15728: added test exclusion for runBlocking check

* For #15278: changed null check and added Synchronized to count setter

* For #15278: fix for nits

* For #15278: added StartupExcessiveResourceUseTest to CODEOWNERS

* For #15278: fixed for nits

* For #15278: Moved increment function to extension function and fixed indentation

* For #15278: Added tests for Atomic Integer extension and nit fix
2020-11-02 12:49:17 -05:00
Michael Comella
25d3ea65f5 For #15707: add ConstraintLayoutPerfDetectorTest to code owners. 2020-10-30 18:23:35 -07:00
Michael Comella
c071889520 For #15280: add new lint rule to CODE OWNERS. 2020-10-14 15:20:28 -07:00
Michael Comella
b222376cb0 For #13959: add test to prevent increasing StrictMode suppression count. 2020-10-06 14:11:53 -07:00
Roger Yang
450caf1e42
Revert "No issue: remove * ac from codeowners." (#15638)
This reverts commit f3b7e294be.
2020-10-02 13:08:15 -04:00
Michael Comella
f3b7e294be No issue: remove * ac from codeowners.
rocketsroger and I agreed this may be undesireable due to the increased
notifications. If we remove it and someone complains, we can re-add it.
2020-10-01 17:09:39 -07:00
Michael Comella
e04e20234d For #15273: add warnings about codeowners; add perf team as CODEOWNERS owner. 2020-09-24 08:33:18 -07:00
Michael Comella
70df3e8b15 For #15273: fix suspected syntax errors in code owners.
I validated the paths used using a technique in this blog post:
http://www.benjaminoakes.com/git/2018/08/10/Testing-changes-to-GitHub-CODEOWNERS/

So the syntax error(s) are likely within the names, such as this
example.
2020-09-24 08:33:18 -07:00
Michael Comella
88a539ee5a No issue: monitor performance via CODEOWNERS. 2020-08-26 15:48:54 -07:00
Johan Lorenzo
1d30f313d3 Add Releng and Fenix as code owners of taskcluster/, .taskcluster.yml, .cron.yml 2019-09-26 09:36:06 -07:00
Colin Lee
ca118033a2
Updated CODEOWNERS file to let A-C review Fenix PRs (#5013) 2019-08-29 14:26:16 -05:00
Mitchell Hentges
68756300c8 Ensures that fenix team is also notified of automation changes 2019-02-15 14:10:39 -08:00
Mitchell Hentges
8293f6a7a7 Changes global code owner to mozilla-mobile/fenix 2019-02-15 12:51:44 -08:00
Mitchell Hentges
31be66bd6f Sets up CODEOWNERS file 2019-02-15 12:51:44 -08:00