Closed
Bug 1615884
Opened 5 years ago
Closed 4 years ago
clang-format mercurial extension won't respect .clang-format-ignore
Categories
(Developer Infrastructure :: Lint and Formatting, defect, P3)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox84 fixed)
RESOLVED
FIXED
84 Branch
Tracking | Status | |
---|---|---|
firefox84 | --- | fixed |
People
(Reporter: emk, Assigned: padenot)
References
Details
Attachments
(1 file)
Steps to reproduce:
- Enable clang-format extension (
./mach vcs-setup
will do it for you.) - Modify a test file under
build/clang-plugin/tests
. - Create a new changeset:
hg commit
Actual result:
clang-format extension will format the modified file and the test will break.
Expected result:
clang-format extension should not touch the file.
Reporter | ||
Updated•5 years ago
|
Type: task → defect
Updated•5 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
Assignee: nobody → padenot
Status: NEW → ASSIGNED
Pushed by padenot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f9a15d3564b9
Handle Windows separators correctly when calling clang-format -d on Windows, while allowing writing path with '/' on the command-line. r=andi
Comment 3•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox84:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•