Commit Graph

13 Commits (3e14d2215b7e83adf36d543021e279370882d2e8)

Author SHA1 Message Date
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 6 years ago
Colin Lee c0a89dc3c1 [fenix] Make ktlint and detekt style changes 6 years ago