2
0
mirror of https://github.com/fork-maintainers/iceraven-browser synced 2024-11-15 18:12:54 +00:00
Commit Graph

26 Commits

Author SHA1 Message Date
Michael Comella
9d763280e9 Closes #18164: on pre-push, check for uncommitted glean docs. 2021-02-24 15:44:39 -08:00
Michael Comella
0f770ca39b For #15279: add MozillaUseLazyMonitored custom detekt rule. 2020-11-05 07:48:17 -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
e246854309 For #15707: add lint rules tests to pre-push hook. 2020-10-30 18:23:35 -07:00
Michael Comella
d2467f7541 For #13959 - post: ignore new StrictMode rule in tests. 2020-10-06 14:11:53 -07:00
Michael Comella
3b5d6d58d9 For #13959: add MozillaStrictModeSuppression detekt check. 2020-10-06 14:11:53 -07:00
Michael Comella
f690f8736d No issue: replace LintUnitTestRunner with detekt implementation.
This should be more performant and easier to maintain.
2020-10-01 16:06:53 -07:00
Kainalu Hagiwara
52d4ffdef0 For #13424 - Fix detekt config and warnings. 2020-08-12 09:04:24 -07:00
Matthew Finkel
b0c187b7e7 For #13363: Update pre-push-recommended's flavor 2020-08-06 14:33:51 -07:00
Tiger Oakes
61d6b333de Update LongParameterList baseline 2020-07-27 14:20:34 -04:00
Tiger Oakes
52de2b8f38 Fix license on remaining files 2020-07-27 13:04:55 -04:00
liuche
09d0688e64
CI for PR #9705 (#12748)
* #3880. Update detekt to 1.9.1

* #3880. Use `AbsentOrWrongFileLicense` detekt rule

* #3880. Update detekt baseline

Co-authored-by: Denys M <dector9@gmail.com>
2020-07-20 15:09:58 -07:00
Denys M
ff3e738794 #12169. Fix pre-push hook instructions. 2020-07-01 23:02:32 -04:00
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>
2020-04-01 15:42:46 -07:00
Michael Comella
c0d7a1f759 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.
2020-03-19 17:24:22 -07:00
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.
2020-01-17 15:46:32 -08:00
Michael Comella
7d754cc60e 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.
2019-12-09 22:47:00 +01:00
Jeff Boek
68f5d211d3 For #4326 - Updates codebase to support latest version of detekt 2019-08-21 13:50:51 -07:00
Mitchell Hentges
6c020a0bc4 Configure either geckoview beta or nightly at compile-time (#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
2019-08-21 08:32:01 -07:00
Tiger Oakes
4994554576 No issue: Adds lint exception for when statement (#3685) 2019-06-26 15:52:24 -07:00
Mitchell Hentges
92b6d4a925 Moves release flavour to build type (#1693) 2019-04-22 11:02:39 -07:00
ekager
052ab2383f No issue: Add instructions to pre-push hook 2019-04-19 13:46:25 -07:00
Sebastian Kaspari
0bffc071b6 Closes #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).
2019-02-27 13:02:44 -06:00
Sebastian Kaspari
9876970b16 Mark pre-push-recommended as executable. 2019-01-31 09:03:30 -08:00
Colin Lee
8af55652be Add project README and pre-push-hook 2019-01-30 10:32:02 -08:00
Colin Lee
b2b6a530d0 Make ktlint and detekt style changes 2019-01-30 11:08:25 -06:00