Page MenuHomePhabricator
Feed Advanced Search

Jan 25 2016

pwithnall added a comment to T10219: Show suggested lint autofix in Differential web view.

Can you give me an example of a rule which is sometimes wrong and where the implied patch is not obvious from the rule name?

Jan 25 2016, 5:47 PM · Lint, Differential
pwithnall updated subscribers of T10219: Show suggested lint autofix in Differential web view.
Jan 25 2016, 5:26 PM · Lint, Differential
pwithnall created T10219: Show suggested lint autofix in Differential web view.
Jan 25 2016, 5:26 PM · Lint, Differential

Jul 1 2015

pwithnall added a comment to T8709: arc diff: option to just add Differential ref to commit message (middle ground between mutable and immutable workflows).

I don’t want to change my project’s established conventions and clutter my git logs because arc wants to scatter droppings all over my commit messages.

It sounds like Phabricator isn't a very good fit for your project. You can find some alternatives on Wikipedia.

Jul 1 2015, 5:43 PM · Arcanist
pwithnall added a comment to T8709: arc diff: option to just add Differential ref to commit message (middle ground between mutable and immutable workflows).

Another example would be the ‘Summary:’ heading — git commit message formatting already defines what the summary and body of the commit message are.

Jul 1 2015, 5:34 PM · Arcanist
pwithnall added a comment to T8709: arc diff: option to just add Differential ref to commit message (middle ground between mutable and immutable workflows).

I second this request, although if it could be fixed in a way which avoids adding another configuration option, that would be great (I agree with T8227).

Jul 1 2015, 3:08 PM · Arcanist