Page MenuHomePhabricator

ezyang (Edward Z. Yang)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Oct 2 2014, 8:30 PM (532 w, 5 d)
Availability
Available

Recent Activity

Apr 18 2017

ezyang updated Q610: What is arc diff --verbatim supposed to do, actually? from to What is arc diff --verbatim supposed to do, actually?.
Apr 18 2017, 3:27 PM

Aug 5 2015

ezyang added a comment to T6050: Improve error message / UX when fields are specified twice in revision information.

I've run into this problem by copy pasting my commit message into the description field so that I can update the description when my commit updates. If I also copy in the metadata fields, this breaks the parser, e.g.

Aug 5 2015, 8:02 PM · Differential

Oct 16 2014

ezyang added a comment to T6323: Provide UI indication you've reached the end of a Diff's comments.

Most search & find interfaces give a visual cue when they wrap around, however. Would a patch along those lines be accepted?

Oct 16 2014, 6:51 PM · Design, Differential
ezyang updated the task description for T6324: Next should jump to line immediately above comment, rather than comment itself.
Oct 16 2014, 5:50 AM · Differential
ezyang created T6324: Next should jump to line immediately above comment, rather than comment itself.
Oct 16 2014, 5:50 AM · Differential
ezyang created T6323: Provide UI indication you've reached the end of a Diff's comments.
Oct 16 2014, 5:39 AM · Design, Differential
ezyang added a comment to T1591: Differential "buoyant" header has been temporarily disabled.

Could we solve the problem of knowing what file a patch is for without scrolling just by adding a tooltip to, say, the line numbers? (Even the browser tooltip would work.) It doesn't have to be immediately visible, I just want to find out the file without scrolling!

Oct 16 2014, 5:30 AM · Inline Comments, Differential

Oct 6 2014

ezyang added a comment to T5636: add option to arc diff to include entire commit history for review.

Thank you for the pointer on history mutability!

Oct 6 2014, 5:41 PM · Arcanist
ezyang added a comment to T5636: add option to arc diff to include entire commit history for review.

I'd also like to chime in that it is frustrating to (1) make sure all of patches in the series have correct dependency information (T6232), and (2) that the Phabricator web interface doesn't have any affordances for handling patch series like this (e.g. collecting all the patches in a logical series into a single summary page.)

Oct 6 2014, 4:14 PM · Arcanist

Oct 2 2014

ezyang added a subtask for T6236: arc diff should support automatically splitting into multiple commits: T6232: arc diff should suggest/pre-fill dependencies.
Oct 2 2014, 11:05 PM · Arcanist
ezyang created T6236: arc diff should support automatically splitting into multiple commits.
Oct 2 2014, 11:04 PM · Arcanist
ezyang added a comment to T6233: Phabricator web UI is unusable when patch gets too big.

Sorry for all the noise, but is this WONTFIX for having Arcanist warn when the diff is really big?

Oct 2 2014, 10:56 PM · Phabricator
ezyang updated the task description for T6234: Add option to make arc lint interactive.
Oct 2 2014, 10:33 PM · Arcanist
ezyang added a comment to T6235: Text lint checker for tabs should only check changed lines.

To clarify, the severity determines whether or not the error is reported always, or only if it was touched by the commit.

Oct 2 2014, 10:01 PM · Arcanist
ezyang created T6235: Text lint checker for tabs should only check changed lines.
Oct 2 2014, 9:11 PM · Arcanist
ezyang created T6234: Add option to make arc lint interactive.
Oct 2 2014, 9:07 PM · Arcanist
ezyang created T6233: Phabricator web UI is unusable when patch gets too big.
Oct 2 2014, 8:49 PM · Phabricator
ezyang added a project to T6231: All parameters for arc diff should be passable as command line flags: Arcanist.
Oct 2 2014, 8:33 PM · Arcanist
ezyang created T6232: arc diff should suggest/pre-fill dependencies.
Oct 2 2014, 8:33 PM · Arcanist
ezyang created T6231: All parameters for arc diff should be passable as command line flags.
Oct 2 2014, 8:32 PM · Arcanist