Bug 1891723 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Using the command `./mach lint --list`, you can see all of the linters available in AC.

Several of these linters prefixed by `android` were historically GeckoView only linters:

```
android-api-lint
android-checkstyle
android-format
android-javadoc
android-lint
android-test
```

This bug is to consider renaming these to be prefixed by `geckoview` and/or repurpose to include AC/Fenix.

Right now to run the core AC/Fenix lints, the developer must CD into `.../mobile/android/fenix` or `.../mobile/android/android-components` and run `./gradlew ktlintFormat` and `./gradlew detekt`.
Revision 1 by
on 
Using the command `./mach lint --list`, you can see all of the linters available in MC.

Several of these linters prefixed by `android` were historically GeckoView only linters:

```
android-api-lint
android-checkstyle
android-format
android-javadoc
android-lint
android-test
```

This bug is to consider renaming these to be prefixed by `geckoview` and/or repurpose to include AC/Fenix.

Right now to run the core AC/Fenix lints, the developer must CD into `.../mobile/android/fenix` or `.../mobile/android/android-components` and run `./gradlew ktlintFormat` and `./gradlew detekt`.
Revision 2 by
on 
Using the command `./mach lint --list`, you can see all of the linters available in MC.

Several of these linters prefixed by `android` were historically GeckoView only linters:

```
android-api-lint
android-checkstyle
android-format
android-javadoc
android-lint
android-test
```

This bug is to consider renaming these to be prefixed by `geckoview` and/or repurpose to include AC/Fenix.

Right now to run the core AC/Fenix lints, the developer must CD into `.../mobile/android/fenix` or `.../mobile/android/android-components` and run `./gradlew ktlintFormat` and `./gradlew detekt`.

Using `./mach lint` will help developers familiar with AC, but not Android to contribute and standardize linters.
Revision 3 by
on 
Using the command `./mach lint --list`, you can see all of the linters available in MC.

Several of these linters prefixed by `android` were historically GeckoView only linters:

```
android-api-lint
android-checkstyle
android-format
android-javadoc
android-lint
android-test
```

This bug is to consider renaming these to be prefixed by `geckoview` and/or repurpose to include AC/Fenix.

Right now to run the core AC/Fenix lints, the developer must CD into `.../mobile/android/fenix` or `.../mobile/android/android-components` and run `./gradlew ktlintFormat` and `./gradlew detekt`.

Using `./mach lint` will help developers familiar with MC, but not Android, to contribute and standardize linters.

Back to Bug 1891723 Comment 0