Commit Graph

25 Commits (e9c9154b3316e0fe6efc1902ea086e6fff8c0b69)

Author SHA1 Message Date
Michael Comella e0ee4f6d40 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15279: add MozillaUseLazyMonitored custom detekt rule. 4 years ago
MarcLeclair 78850c7e3d [fenix] 15278 detekt rule runblocking (https://github.com/mozilla-mobile/fenix/pull/15942)
* For https://github.com/mozilla-mobile/fenix/issues/15278: added CoroutineManager to count runBlocking calls

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added actual detekt rule for runblocking and its config to the yaml

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added unit test for RunblockingCounter

* For https://github.com/mozilla-mobile/fenix/issues/15278: renamed StrictModeStartupSuppressionCountTest.kt to PerformanceStartupTest.kt and added runBlockingCount test

* Lint fix

* For https://github.com/mozilla-mobile/fenix/issues/15278: made runblocking a Long to prevent overflow

* For https://github.com/mozilla-mobile/fenix/issues/15278: fixed MozRunblocking name, description and moved RunBlockingCounter to perf package

* For https://github.com/mozilla-mobile/fenix/issues/15278:Renamed MozillaRunblockingCheck to MozillaRunBlockingCheck

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added setup for unit test, since it failed without restting counter

* For https://github.com/mozilla-mobile/fenix/issues/15278: Fixed naming for RunBlocking lint check

* For https://github.com/mozilla-mobile/fenix/issues/15278: removed changes made to test to use runBlockingIncrement

* For https://github.com/mozilla-mobile/fenix/issues/15728: added test exclusion for runBlocking check

* For https://github.com/mozilla-mobile/fenix/issues/15278: changed null check and added Synchronized to count setter

* For https://github.com/mozilla-mobile/fenix/issues/15278: fix for nits

* For https://github.com/mozilla-mobile/fenix/issues/15278: added StartupExcessiveResourceUseTest to CODEOWNERS

* For https://github.com/mozilla-mobile/fenix/issues/15278: fixed for nits

* For https://github.com/mozilla-mobile/fenix/issues/15278: Moved increment function to extension function and fixed indentation

* For https://github.com/mozilla-mobile/fenix/issues/15278: Added tests for Atomic Integer extension and nit fix
4 years ago
Michael Comella 62a0f6b088 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15707: add lint rules tests to pre-push hook. 4 years ago
Michael Comella 4e7658e479 [fenix] For https://github.com/mozilla-mobile/fenix/issues/13959 - post: ignore new StrictMode rule in tests. 4 years ago
Michael Comella 19904397b2 [fenix] For https://github.com/mozilla-mobile/fenix/issues/13959: add MozillaStrictModeSuppression detekt check. 4 years ago
Michael Comella e1ffe1108f [fenix] No issue: replace LintUnitTestRunner with detekt implementation.
This should be more performant and easier to maintain.
4 years ago
Kainalu Hagiwara e17c4e5f90 [fenix] For https://github.com/mozilla-mobile/fenix/issues/13424 - Fix detekt config and warnings. 4 years ago
Matthew Finkel ccdfa79623 [fenix] For https://github.com/mozilla-mobile/fenix/issues/13363: Update pre-push-recommended's flavor 4 years ago
Tiger Oakes 233708b122 [fenix] Update LongParameterList baseline 4 years ago
Tiger Oakes ad4476342b [fenix] Fix license on remaining files 4 years ago
liuche 8932298651 [fenix] CI for PR https://github.com/mozilla-mobile/fenix/pull/9705 (https://github.com/mozilla-mobile/fenix/pull/12748)
* https://github.com/mozilla-mobile/fenix/issues/3880. Update detekt to 1.9.1

* https://github.com/mozilla-mobile/fenix/issues/3880. Use `AbsentOrWrongFileLicense` detekt rule

* https://github.com/mozilla-mobile/fenix/issues/3880. Update detekt baseline

Co-authored-by: Denys M <dector9@gmail.com>
4 years ago
Denys M 4261dcf907 [fenix] https://github.com/mozilla-mobile/fenix/issues/12169. Fix pre-push hook instructions. 4 years ago
Emily Kager 522aca44f4 [fenix] Upgrade to detekt 1.6.0 (https://github.com/mozilla-mobile/fenix/pull/9306)
Changelog:

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

Fixes https://github.com/mozilla-mobile/fenix/issues/7360.  Also use more exact test glob.

Co-authored-by: Andrew Gaul <andrew@gaul.org>
4 years ago
Michael Comella b2cd47accd [fenix] No issue: pre-push: assemble everything before testing.
It's better to fail fast on pre-push hooks so devs can get feedback as
fast as possible. Assembling is faster than unit testing so we assemble
everything first before unit testing.
4 years ago
Will Hawkins 78e86d8f2b [fenix] 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.
5 years ago
Michael Comella 25fdcb5596 [fenix] No issue: make pre-push hook assemble only one variant, run tests, build UI test.
Since it's redundant to the definition in the source, I intend to remove
the pre-push hook listed in the wiki after this merges.
5 years ago
Jeff Boek 6ec0801d0c [fenix] For https://github.com/mozilla-mobile/fenix/issues/4326 - Updates codebase to support latest version of detekt 5 years ago
Mitchell Hentges e7ca5257b4 [fenix] Configure either geckoview beta or nightly at compile-time (https://github.com/mozilla-mobile/fenix/pull/4851)
* Remove "abi" product flavor and introduce "engine" product flavor.

This patch will allow us to build Fenix against GeckoView Nightly and GeckoView Beta by
introducing a new flavor dimension: engine = [geckoNightly, geckoBeta].

In addition to that it adds a "fenix"  prefix to the nightly, beta and production flavors
to reduce the ambiguity between fenix beta/nightly and GeckoView beta/nightly.

For now the build types have the following engine variants enabled:

**debug**: geckoNightly, geckoBeta
Both variants enabled for local development and testing.

**forPerformanceTest**: geckoNightly, geckoBeta
Both variants enabled unless the perf team only cares about Nightly (tbd)

**fenixNightlyLegacy**: geckoBeta
Uses GeckoView Beta for now - the same version we ship production builds with (same behavior
as before). This release type will eventualyl be decommissioned once we switch to a separate
Nightly app on Google Play.

**fenixNightly**: geckoBeta
Uses GeckoView Beta for now - the same version we ship production builds with (same behavior
as before). Changing this build to use GeckoView Nightly is currently being discussed.

**fenixBeta**: geckoBeta
Fenix Beta uses GeckoView Beta.

**fenixProduction**
Fenix Production uses GeckoView Beta (69) currently.

* gradle.py/variant.py: Replace "abi" with "engine".

* Disable enableUnitTestBinaryResources until we can switch to Android Gradle plugin 3.5.

* Fenix nightly should use both geckoview nightly and beta

* Updates automation to use apk splitting and support different engine
5 years ago
Tiger Oakes 43248d95e3 [fenix] No issue: Adds lint exception for when statement (https://github.com/mozilla-mobile/fenix/pull/3685) 5 years ago
Mitchell Hentges d2ea8148c8 [fenix] Moves release flavour to build type (https://github.com/mozilla-mobile/fenix/pull/1693) 5 years ago
ekager 20e2908677 [fenix] No issue: Add instructions to pre-push hook 5 years ago
Sebastian Kaspari 5279c3edef [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/698: Create product flavors for Fennec transition.
* greenfield (pure Fenix without anything added)
* firefoxNightly (To replace Firefox Nightly)
* firefoxBeta (To replace Firefox Beta)
* firefoxRelease (To replace Firefox Release)

For now it will be a bit annoying to have all those flavors. But once the transition has been
successfully completed we can start to tear them down again.

Once GeckoView universal builds are available we can get rid of the "abi" dimension.

If this is going to slow down the CI process then I might look into parallelizing the build
or potentially introducing a taskcluster CI setup (if that makes sense).
5 years ago
Sebastian Kaspari a82dc81717 [fenix] Mark pre-push-recommended as executable. 5 years ago
Colin Lee ec0ada616b [fenix] Add project README and pre-push-hook 5 years ago
Colin Lee c0a89dc3c1 [fenix] Make ktlint and detekt style changes 5 years ago