Page MenuHomePhabricator

asteinlein (Anders Steinlein)
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
Sep 30 2014, 1:08 PM (255 w, 19 h)
Availability
Available

Recent Activity

Jul 16 2016

asteinlein added a comment to T11188: Updating to latest Phabricator caused feed corruption.

Just wanted to chime in that I've just encountered this as well. Just did an upgrade, and hundreds (possibly thousands?) of old commits are somehow being reprocessed and are pushed to the feed. It's been processing for a couple of hours now, no idea how much work remains....

Jul 16 2016, 1:45 AM · Bug Report

Mar 15 2016

asteinlein added a comment to T5568: Support `.arcunit`, similar to `.arclint`.

Yeah, I understand, although too bad if that task is as low-pri and as large an undertaking as it seems. :/

Mar 15 2016, 12:05 AM · Unit, Arcanist

Mar 14 2016

asteinlein added a comment to T5568: Support `.arcunit`, similar to `.arclint`.

Any progress on this?

Mar 14 2016, 11:41 PM · Unit, Arcanist

Nov 10 2015

asteinlein added a comment to T5568: Support `.arcunit`, similar to `.arclint`.

Any chance this could be speeded up if we were to sponsor some of the development on this?

Nov 10 2015, 7:24 PM · Unit, Arcanist
asteinlein added a comment to T5568: Support `.arcunit`, similar to `.arclint`.

Big yes, please to this. I came across this task when looking into a way to exclude our Python tests when creating diffs when only our front-end JS code has been touched, for instance. It seems this feature will allow just that?

Nov 10 2015, 5:55 PM · Unit, Arcanist

Jun 8 2015

asteinlein added a comment to T8463: Username `@mentions` prevent creation of revisions with "data not attached" error.

Do the problematic revisions mention a user with @username in the summary or test plan?

Jun 8 2015, 1:36 PM · Restricted Project
asteinlein added a comment to T8463: Username `@mentions` prevent creation of revisions with "data not attached" error.

Seems to work fine, which doesn't suprise me ref. my last comment...

Jun 8 2015, 1:23 PM · Restricted Project
asteinlein retitled D13204: Testing new revision from to Testing new revision.
Jun 8 2015, 1:22 PM
asteinlein added a revision to T8463: Username `@mentions` prevent creation of revisions with "data not attached" error: D13204: Testing new revision.
Jun 8 2015, 1:22 PM · Restricted Project
asteinlein added a comment to T8463: Username `@mentions` prevent creation of revisions with "data not attached" error.

How should I do that? But we have further info. A collegue of mine checked out my branch, and successfully created the diff I was unable to. So this seems to be specific to my user...

Jun 8 2015, 1:17 PM · Restricted Project
asteinlein added a comment to T8463: Username `@mentions` prevent creation of revisions with "data not attached" error.

FYI, I also created creating the diff manually through Differential on the web interface. Same exception occured.

Jun 8 2015, 12:47 PM · Restricted Project
asteinlein triaged T8463: Username `@mentions` prevent creation of revisions with "data not attached" error as Unbreak Now! priority.

I'm sorry for bumping this, but I'm actually totally stuck, and can't seem to find any way to create revisions. We just upgraded Phabricator again to see if this had been resolved since our initial upgrade this weekend, but no go...

Jun 8 2015, 12:16 PM · Restricted Project
asteinlein added a comment to T8463: Username `@mentions` prevent creation of revisions with "data not attached" error.

I think we're bitten by the same issue, although nothing to do with Conduit on our end as far as I can tell. Creating a new revision with arc diff crashes for me today, and that is after we upgraded Phabricator this weekend. Stack trace:

Jun 8 2015, 12:13 PM · Restricted Project

May 15 2015

asteinlein added a comment to T7447: Bring inline comments forward across revision updates.

Most of the issues our team run into stem from the fact that we can't toggle the comments temporarily in order to get a distraction free view of the code. Some of our inlines spark some quite lengthy discussions which can sometimes get in the way when you only need to see the code.
We do use the diff of diffs feature heavily which helps this, but it's not always an option for a reviewer due to the fact we generally require two accepts on a diff before it lands. It's not uncommon for a diff to be in a second or third iteration by the time a second reviewer starts their own review from scratch. For the second reviewer, seeing previous discussions isn't always useful - it can be, but we also find that often it's desirable just to hide the noise for a minute to view the current code in its entirety before continuing with the review.

May 15 2015, 10:19 AM · Prioritized, Differential

Apr 30 2015

asteinlein added a comment to T7447: Bring inline comments forward across revision updates.

I don't know @epriestley, I found it really confusing to have my comment on a different line entirely.

+1

Apr 30 2015, 10:36 PM · Prioritized, Differential
asteinlein added a comment to T7447: Bring inline comments forward across revision updates.

I'm sorry to say I don't for the life of me understand how "this feature (as it exists today) is a net improvement to Differential". See this screenshot from one of our reviews, which both hightlights the fact that the diff is terribly difficult to read, and even more important, has major issues with where the forward-moving comments appear:

Apr 30 2015, 10:32 PM · Prioritized, Differential
asteinlein added a comment to T7447: Bring inline comments forward across revision updates.

As a reviewer, I think bringing "Done" comments forward is quite important: it makes it a lot easier for me to verify that they're really done. This is the #1 benefit I've reaped from this feature so far.

Apr 30 2015, 11:40 AM · Prioritized, Differential

Apr 27 2015

asteinlein added a comment to T1460: Add "Done" link to an inline comment.

The Done state should maybe propagate to the replies of the comment?

Very much +1 to this.

Apr 27 2015, 7:34 PM · User Delight, Differential
asteinlein added a comment to T7447: Bring inline comments forward across revision updates.

We're liking the new feature to mark inline comments as done, however, we dislike the change that makes old comments "stick" to updated diffs. This makes it much harder to read the new updated diff when there are lots of inline comments. Any plans to improve this somehow?

Can you explain your use case a little more? Why are there lots of inline comments? (See discussion earlier in this task.)

Apr 27 2015, 7:24 PM · Prioritized, Differential

Apr 26 2015

asteinlein added a comment to T1460: Add "Done" link to an inline comment.

The Done state should maybe propagate to the replies of the comment?

Apr 26 2015, 6:07 PM · User Delight, Differential
asteinlein added a comment to T1460: Add "Done" link to an inline comment.

We're liking the new feature to mark inline comments as done, however, we dislike the change that makes old comments "stick" to updated diffs. This makes it much harder to read the new updated diff when there are lots of inline comments. Any plans to improve this somehow?

Apr 26 2015, 6:03 PM · User Delight, Differential

Sep 30 2014

asteinlein added a comment to T5689: Batch edit of Maniphest custom fields.

I haven't contributed anything to Phabricator yet, but this is a feature we need, so I might be able to take a stab at this. Any feedback whether this is something that will be accepted, and how complicated it might be?

Sep 30 2014, 1:10 PM · Custom Fields, Maniphest