Page MenuHomePhabricator
Feed Advanced Search

Jan 6 2018

cspeckmim awarded Changelog a Mountain of Wealth token.
Jan 6 2018, 4:47 PM

Oct 14 2016

Pawka awarded T10189: Differential email headers seem to be missing a Like token.
Oct 14 2016, 6:53 AM · Differential, Mail, Bug Report

Jul 12 2016

edibiase added a comment to T11291: Error while executing task (phd daemon).

It's possible that we're in the wrong by falling back to set character set utf8;.

For anyone who can reproduce this, does this fail?

mysql> set character set utf8mb4;
Jul 12 2016, 2:47 PM · Daemons, Bug Report

Jul 9 2016

edibiase added a comment to T11291: Error while executing task (phd daemon).

@eadler, I was just able to resolve this on our installation by installing the php56w-mysqlnd package instead of php56w-mysql, which is what we had installed before.

Jul 9 2016, 6:48 PM · Daemons, Bug Report

May 23 2016

edibiase added a comment to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.

That sounds like it might be what's going on. I apologize for not having better steps to reproduce; I'm working backward from example emails that folks are receiving, so it's been a bit of a guessing game as to how to make this happen.

May 23 2016, 9:00 PM · Owners, Bug Report
edibiase created T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.
May 23 2016, 8:32 PM · Owners, Bug Report

Mar 7 2016

edibiase added a comment to T10511: arc browse doesn't percent encode slashes in branch names.

Awesome; thank you!

Mar 7 2016, 3:14 PM · Arcanist, Bug Report

Mar 3 2016

edibiase created T10511: arc browse doesn't percent encode slashes in branch names.
Mar 3 2016, 7:38 PM · Arcanist, Bug Report

Feb 19 2016

edibiase updated the task description for T10393: Differential revision's images cannot be viewed as raw files.
Feb 19 2016, 5:57 PM · Restricted Project, Differential, Files, Bug Report
edibiase created T10393: Differential revision's images cannot be viewed as raw files.
Feb 19 2016, 5:56 PM · Restricted Project, Differential, Files, Bug Report

Jan 22 2016

edibiase added a comment to T10109: Differental does not trigger Herald for mundane updates.

Thanks, @epriestley (and sorry for getting back to you so late)! You're totally right that this is because I didn't actually update the revision.

Jan 22 2016, 3:47 PM · Differential, Restricted Project, Herald, Bug Report

Jan 20 2016

edibiase created T10189: Differential email headers seem to be missing.
Jan 20 2016, 8:19 PM · Differential, Mail, Bug Report

Jan 9 2016

edibiase created T10109: Differental does not trigger Herald for mundane updates.
Jan 9 2016, 2:45 PM · Differential, Restricted Project, Herald, Bug Report

Dec 9 2015

edibiase created T9940: Revision Project Reviewers state not updated after reviewer added to project.
Dec 9 2015, 4:03 PM · Bug Report

Nov 10 2015

edibiase created T9758: Diff context for Objective-C doesn't show method name if open brace is on newline.
Nov 10 2015, 8:42 PM · Differential

Nov 9 2015

edibiase added a comment to T9741: Default Edit Policy for Differential.

Excellent; that's good to know. Thanks!

Nov 9 2015, 4:35 PM · Differential
edibiase added a comment to T9741: Default Edit Policy for Differential.

Are these updates legitimately accidental? Can you explain how users are accidentally making updates to others' revisions? I don't think we've seen this behavior on this install, or heard about it from other installs.

Nov 9 2015, 4:29 PM · Differential
edibiase created T9741: Default Edit Policy for Differential.
Nov 9 2015, 3:50 PM · Differential

Nov 6 2015

edibiase added a comment to T9727: Show Submit button once a comment has been added to a revision.

Thank you—that makes total sense. I'll track any feedback I hear from folks but will hold off on sending it your way until an appropriate feedback window opens up.

Nov 6 2015, 3:51 PM · Differential
edibiase created T9727: Show Submit button once a comment has been added to a revision.
Nov 6 2015, 3:18 PM · Differential

Nov 4 2015

edibiase created T9713: In Diffusion, show commit messages as parsed fields by default with an option to view them raw.
Nov 4 2015, 8:08 PM · Diffusion (v3)

Nov 2 2015

edibiase added a comment to T9686: Ability to hide inline comments.

Cool; that makes total sense. Thanks again!

Nov 2 2015, 4:35 PM · Differential
edibiase added a comment to T9686: Ability to hide inline comments.

Thanks!

Nov 2 2015, 4:31 PM · Differential
edibiase added a comment to T9687: Issue tracker for revisions.

Thanks!

Nov 2 2015, 4:30 PM · Differential
edibiase added a comment to T9688: Option to show Real Name instead of username.

Thanks!

Nov 2 2015, 4:12 PM
edibiase created T9688: Option to show Real Name instead of username.
Nov 2 2015, 4:05 PM
edibiase created T9687: Issue tracker for revisions.
Nov 2 2015, 3:55 PM · Differential
edibiase created T9686: Ability to hide inline comments.
Nov 2 2015, 3:38 PM · Differential

Oct 29 2015

edibiase added a comment to T9661: arc land doesn't return to local tracking branch if it has a different name from the remote branch it's tracking.

This works great now; thank you very much!

Oct 29 2015, 4:24 PM · Arcanist

Oct 28 2015

edibiase added a comment to T9661: arc land doesn't return to local tracking branch if it has a different name from the remote branch it's tracking.

oh come on you're just making up new workflows at this point

Oct 28 2015, 7:33 PM · Arcanist
edibiase created T9661: arc land doesn't return to local tracking branch if it has a different name from the remote branch it's tracking.
Oct 28 2015, 5:50 PM · Arcanist
edibiase added a comment to T9660: arc land does not delete landed branch when landing directly to remote.

This is working great now; thanks so much for the fast turnaround!

Oct 28 2015, 5:14 PM · Arcanist
edibiase added a comment to T9660: arc land does not delete landed branch when landing directly to remote.

That makes total sense—deleting master seems like a really bad idea in that case.

Oct 28 2015, 4:49 PM · Arcanist
edibiase added a comment to T9658: arc land doesn't default --onto to a tracked branch if it is a remote.

Awesome; D14357 does seem to fix this. Thanks! (Also, the new help documentation for arc land looks great.)

Oct 28 2015, 4:29 PM · Arcanist
edibiase created T9660: arc land does not delete landed branch when landing directly to remote.
Oct 28 2015, 4:27 PM · Arcanist
edibiase created T9658: arc land doesn't default --onto to a tracked branch if it is a remote.
Oct 28 2015, 3:27 PM · Arcanist

Oct 25 2015

CaptSpot awarded Changelog a Like token.
Oct 25 2015, 9:38 AM

Oct 23 2015

vinzent awarded Changelog a Like token.
Oct 23 2015, 9:26 AM

Oct 14 2015

edibiase added a comment to T9562: Show only the first N branches/tags on commits in Diffusion, with a "Show all..." link.

I'd be inclined to show N branches, for some small value of N like 3 or 5 or 10, with a link to "Show 1,695 more branches which contain this commit...".

Preferentially choosing autoclose branches to order first in that N might be desirable, although picking the first N that the VCS gives us may be faster or more scalable (e.g., if you had 170,000 instead of 1,700, the cost to select the "good" ones might not be worth paying).

Oct 14 2015, 12:38 AM · Diffusion (v3), Restricted Project

Oct 13 2015

edibiase added a comment to T9562: Show only the first N branches/tags on commits in Diffusion, with a "Show all..." link.

A nice middle ground would be if I could specify to show some branches by default—for example, just autoclose branches—and to have others hidden unless specifically requested.

Oct 13 2015, 11:14 PM · Diffusion (v3), Restricted Project
edibiase created T9562: Show only the first N branches/tags on commits in Diffusion, with a "Show all..." link.
Oct 13 2015, 11:08 PM · Diffusion (v3), Restricted Project

Oct 10 2015

edibiase added a comment to T9535: Option to CC or use a mailing list for project reviewers.

I think you're right; the best way forward for us to is to get more powerful filtering rules on the server side. In the meantime, your suggestion for using a mailing list or separate teams seems like a reasonable workaround. Thanks!

Oct 10 2015, 3:18 PM · Restricted Project, Differential

Oct 9 2015

edibiase added a comment to T9543: In "arc land", recursively resolve tracking branches until we reach one in a remote.

(Cool—that seems very reasonable.)

Oct 9 2015, 8:48 PM · Arcanist
edibiase added a comment to T9543: In "arc land", recursively resolve tracking branches until we reach one in a remote.

It seems perfectly reasonably to tackle the specialty-workflow problem via T5055, rather than blowing up the number of Arcanist flags. And, as you suggest, I think T9537 will largely eliminate my desire for this, since it will allow me to name my local feature branches whatever I want, and I'll only have to type the long remote branch name at feature branch creation time.

Oct 9 2015, 6:46 PM · Arcanist
edibiase created T9543: In "arc land", recursively resolve tracking branches until we reach one in a remote.
Oct 9 2015, 5:43 PM · Arcanist
edibiase added a comment to T9537: Support for landing without local tracking branch.

My primary concern with implementing this as the default/only behavior is that we won't have anywhere to put the user when we're done that makes sense to an inexperienced / novice Git user.

Oct 9 2015, 5:18 PM · Arcanist
edibiase added a comment to T9537: Support for landing without local tracking branch.

Thanks, as always, for your prompt and thoughtful responses!

Oct 9 2015, 3:24 PM · Arcanist

Oct 8 2015

edibiase created T9537: Support for landing without local tracking branch.
Oct 8 2015, 9:06 PM · Arcanist
edibiase added a comment to T9535: Option to CC or use a mailing list for project reviewers.

I think you can create a "Mailing List" user and add them to the project to accomplish this. This is in addition to other mail delivery.

Oct 8 2015, 5:54 PM · Restricted Project, Differential
edibiase added a comment to T9535: Option to CC or use a mailing list for project reviewers.

Thanks! It looks like the message delivery was voided because of my preference not to receive mail from Differential:

Oct 8 2015, 5:18 PM · Restricted Project, Differential
edibiase added a comment to T9535: Option to CC or use a mailing list for project reviewers.

Hrm; I added a Herald rule that would email me if I'm in the list of reviewers, and changed all of the Differential Revision notification preferences to "Notify", but I didn't get an email when I changed a revision where I was a direct reviewer. (I verified that the Herald rule matched the revision in question using the test console.)

Oct 8 2015, 5:02 PM · Restricted Project, Differential
edibiase added a comment to T9535: Option to CC or use a mailing list for project reviewers.

Thanks for the quick reply, @chad! If I understand it correctly, it sounds like you're right—T5791 is a superset of this request. Being able to write a Herald rule that drops outbound mail when I'm a project reviewer but allows it when I'm a direct reviewer would pretty much cover my use case.

Oct 8 2015, 4:51 PM · Restricted Project, Differential
edibiase created T9535: Option to CC or use a mailing list for project reviewers.
Oct 8 2015, 3:44 PM · Restricted Project, Differential

Oct 3 2015

edibiase renamed T9506: arc land's help text doesn't mention default of landing in tracked branch from arc land's help text for the "onto" branch is potentially confusing to arc land's help text doesn't mention default of landing in tracked branch.
Oct 3 2015, 4:24 PM · Documentation, Arcanist
edibiase created T9506: arc land's help text doesn't mention default of landing in tracked branch.
Oct 3 2015, 4:21 PM · Documentation, Arcanist
edibiase created T9505: Show landing branch information in Differential revision detail.
Oct 3 2015, 4:08 PM · Differential

Aug 26 2015

edibiase added a comment to T9268: PREG_BACKTRACK_LIMIT_ERROR parsing commit under PhabricatorRepositoryCommitHeraldWorker/PhabricatorCustomFieldMonogramParser.

Thank you very much! I've updated to HEAD and everything seems to be running smoothly now.

Aug 26 2015, 5:23 PM · Herald
edibiase created T9268: PREG_BACKTRACK_LIMIT_ERROR parsing commit under PhabricatorRepositoryCommitHeraldWorker/PhabricatorCustomFieldMonogramParser.
Aug 26 2015, 3:55 PM · Herald