Commit Graph

23 Commits (fork)

Author SHA1 Message Date
Michael Comella 6107143f16 No issue: remove performance codeowners. 2 years ago
Andrew Halberstadt c1d44d1327 No issue: remove 'mozilla-mobile/releng' as CODEOWNER of /automation and /.github
These directories are specific to Fenix. After a quick scan and chat
with Releng, we don't have knowledge / motivation to be maintainers
here.

Releng will continue to be CODEOWNERS of core taskcluster files though.
2 years ago
Michael Comella 343ff924d7 For #19901: add benchmark.gradle to CODEOWNERS. 3 years ago
Michael Comella 49240b239b For #19436: rm Application and FeatureFlags from perf code owners. 3 years ago
Michael Comella 94bbf90f3c No issue: move TestStrictModeManager to perf sub package. 4 years ago
Michael Comella 99600247bb No issue: move relevant files in :mozilla-lint-rules to perf pkg. 4 years ago
Michael Comella c647679e4d No issue: move relevant :mozilla-detekt-rules to perf/ pkg. 4 years ago
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).
4 years ago
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.
4 years ago
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
4 years ago
Michael Comella 25d3ea65f5 For #15707: add ConstraintLayoutPerfDetectorTest to code owners. 4 years ago
Michael Comella c071889520 For #15280: add new lint rule to CODE OWNERS. 4 years ago
Michael Comella b222376cb0 For #13959: add test to prevent increasing StrictMode suppression count. 4 years ago
Roger Yang 450caf1e42
Revert "No issue: remove * ac from codeowners." (#15638)
This reverts commit f3b7e294be.
4 years ago
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.
4 years ago
Michael Comella e04e20234d For #15273: add warnings about codeowners; add perf team as CODEOWNERS owner. 4 years ago
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.
4 years ago
Michael Comella 88a539ee5a No issue: monitor performance via CODEOWNERS. 4 years ago
Johan Lorenzo 1d30f313d3 Add Releng and Fenix as code owners of taskcluster/, .taskcluster.yml, .cron.yml 5 years ago
Colin Lee ca118033a2
Updated CODEOWNERS file to let A-C review Fenix PRs (#5013) 5 years ago
Mitchell Hentges 68756300c8 Ensures that fenix team is also notified of automation changes 5 years ago
Mitchell Hentges 8293f6a7a7 Changes global code owner to mozilla-mobile/fenix 5 years ago
Mitchell Hentges 31be66bd6f Sets up CODEOWNERS file 5 years ago