Closed Bug 1391233 Opened 2 years ago Closed 2 years ago

[clang-format] Ignore the tests in the reformat

Categories

(Firefox Build System :: Source Code Analysis, enhancement)

enhancement
Not set

Tracking

(firefox57 fixed)

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: sylvestre, Assigned: sylvestre)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Not sure we want to reformat all the tests. I propose that we ignore them
Comment on attachment 8898279 [details]
Bug 1391233 - [clang-format] Ignore the tests in the reformat

https://reviewboard.mozilla.org/r/169652/#review175404
Attachment #8898279 - Flags: review?(gps) → review+
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 30182d62d37f -d d0892f7190ef: rebasing 414594:30182d62d37f "Bug 1391233 - [clang-format] Ignore the tests in the reformat r=gps" (tip)
merging .clang-format-ignore
warning: conflicts while merging .clang-format-ignore! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/58b60ea13641
[clang-format] Ignore the tests in the reformat r=gps
https://hg.mozilla.org/mozilla-central/rev/58b60ea13641
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Product: Core → Firefox Build System
Enabling them in bug 1513205 now
You need to log in before you can comment on or make changes to this bug.