Closed Bug 1622368 Opened 6 years ago Closed 6 years ago

Many linters don't run if only their configuration changes

Categories

(Developer Infrastructure :: Lint and Formatting, defect, P3)

defect

Tracking

(firefox76 fixed)

RESOLVED FIXED
mozilla76
Tracking Status
firefox76 --- fixed

People

(Reporter: standard8, Assigned: gaurijove, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [Lang=python] )

Attachments

(1 file)

We picked this up with an issue on m-c. Bug 1563915 changed rustfmt.yml, and after its landing, we discovered that it had missed updating some files, and the rustfmt task was failing.

Looking at the taskcluster configuration, this task is only run for *.rs files:

https://searchfox.org/mozilla-central/rev/f36cb2af46edd2659f446b7acdb2154e230ee445/taskcluster/ci/source-test/mozlint.yml#268

We should change that task to run for its configuration files as well.

We should also review the rest of the tasks in there, and add their configuration files as well.

See Also: → 1622343

The priority flag is not set for this bug.
:ahal, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(ahal)
Flags: needinfo?(ahal)
Priority: -- → P3
Mentor: sledru
Keywords: good-first-bug
Whiteboard: [Lang=python]
Assignee: nobody → gaurijove
Status: NEW → ASSIGNED
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a9377f3c9dd0 Run linters on configuration changes. r=sylvestre
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
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: