Closed Bug 1769275 Opened 2 years ago Closed 2 years ago

mach eslint only detects and fixes issues once

Categories

(Developer Infrastructure :: Lint and Formatting, task)

Desktop
Windows 10

Tracking

(firefox-esr91 unaffected, firefox100 unaffected, firefox101 unaffected, firefox102 fixed)

RESOLVED FIXED
102 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox100 --- unaffected
firefox101 --- unaffected
firefox102 --- fixed

People

(Reporter: barret, Assigned: barret)

References

(Regression)

Details

Attachments

(1 file)

I'm on Windows 10 using Powershell and mozilla-build 4.

STR:

  1. rm -Recurse -Force node_modules
  2. .\mach eslint --fix path/to/file. The file is fixed up.
  3. hg up -C . The file is reverted to its version with errors.
  4. .\mach eslint --fix path/to/file. The file is unchanged
  5. rm -Recurse -Force node_modules
  6. .\mach eslint --fix path/to/file. The file is fixed up again

ER
I would expect .\mach eslint to work every time.

AR
.\mach eslint only works the first time I run it after blowing away node_modules.

Running .\node_modules\.bin\eslint --fix path/to/file directly does work in the interim.

Regressed by: 1765361

Set release status flags based on info from the regressing bug 1765361

Assignee: nobody → brennie
Status: NEW → ASSIGNED
Attachment #9277375 - Attachment description: Bug 1769275 - Generate topsrcdir with correct casing on Windows in mozbuild r?glandium → Bug 1769275 - Generate topsrcdir with correct casing on Windows in mozbuild r?#build
Pushed by brennie@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b80ee2ab71c4
Generate topsrcdir with correct casing on Windows in mozbuild r=firefox-build-system-reviewers,ahochheiden
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
Has Regression Range: --- → yes
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: