Displaying 3 results from an estimated 3 matches for "t11879".
Did you mean:
11879
2016 Nov 16
2
Highlighting trailing whitespaces on Phab?
So, I forwarded the request for highlighting trailing whitespaces to
phabricator upstream (https://secure.phabricator.com/T11879), and upstream
folks suggest we enable the Lint feature in Arcanist (
https://secure.phabricator.com/book/phabricator/article/arcanist_lint/). This
will enforce the check when `arc diff` is run (reviewers wouldn't see the
warnings though).
There are two linters we might be interested in enabli...
2016 Nov 16
2
Highlighting trailing whitespaces on Phab?
...; tests and show their result in the review as well.
>
> The problem is basically that not many ppl use arcanist...
>
> On 11/16, Eric Liu wrote:
>> So, I forwarded the request for highlighting trailing whitespaces to
>> phabricator upstream (https://secure.phabricator.com/T11879), and upstream
>> folks suggest we enable the Lint feature in Arcanist (
>> https://secure.phabricator.com/book/phabricator/article/arcanist_lint/). This
>> will enforce the check when `arc diff` is run (reviewers wouldn't see the
>> warnings though).
>>
>>...
2016 Nov 16
2
Highlighting trailing whitespaces on Phab?
On 11/14, Mehdi Amini via llvm-dev wrote:
> Ideally I’d even really like to have a both checking for revision on
> phab, clang-formatting them, and post a comment when there is a
> mismatch :)
I'd like that!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: Digital signature
URL: