Bug 272292 Comment 31 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Ping Chen (:rnons) from comment #29)
> Found that `../mach clang-format` inside `comm` folder doesn't work.

Ahh yes - I remember this. It /does/ work inside `comm`, but you have to include `comm\` in the path, eg
```
$ ../mach clang-format -s -p comm/mailnews/....
```
But if you forget the `comm/` it doesn't complain about missing files. It just outputs nothing, which is very confusing.
(In the end, I just hacked up a shell script to run eslint and clang-format for me: https://github.com/bcampbell/tb-notes/blob/master/scripts/cclint)
(In reply to Ping Chen (:rnons) from comment #29)
> Found that `../mach clang-format` inside `comm` folder doesn't work.

Ahh yes - I remember this. It /does/ work inside `comm`, but you have to include `comm/` in the path, eg
```
$ ../mach clang-format -s -p comm/mailnews/....
```
But if you forget the `comm/` it doesn't complain about missing files. It just outputs nothing, which is very confusing.
(In the end, I just hacked up a shell script to run eslint and clang-format for me: https://github.com/bcampbell/tb-notes/blob/master/scripts/cclint)

Back to Bug 272292 Comment 31