Closed Bug 1434332 Opened 6 years ago Closed 6 years ago

Add a checklist for adding new linters to the tree

Categories

(Developer Infrastructure :: Lint and Formatting, enhancement)

3 Branch
enhancement
Not set
normal

Tracking

(firefox60 fixed)

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

As we add new linters to the tree, we should also ensure that we have some basic requirements so that we get everything we need from the outset (or landed in a way which doesn't adversely affect developers).

We can add this into the existing linter documentation.
Comment on attachment 8946702 [details]
Bug 1434332 - Add a checklist for adding new linters to the tree.

https://reviewboard.mozilla.org/r/216672/#review222488

::: tools/lint/docs/create.rst:13
(Diff revision 1)
> +
> +* Any required dependencies should be installed as part of `./mach bootstrap`
> +* A ``./mach lint`` interface
> +* Running ``./mach lint`` command must pass (note, linters can be disabled for individual directories)
> +* Taskcluster/Treeherder integration
> +* In tree documentation (under ``lint/docs``) to give a basic summary, links and any other useful information

nit: tools/lint/docs
Attachment #8946702 - Flags: review?(ahalberstadt) → review+
See Also: → 1434337
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/82352a076662
Add a checklist for adding new linters to the tree. r=ahal
https://hg.mozilla.org/mozilla-central/rev/82352a076662
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Product: Testing → Firefox Build System
Version: Version 3 → 3 Branch
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: