Page MenuHomePhabricator

DifferentialProject
ActivePublic

Details

Description

Differential is a code review tool.

Recent Activity

Tue, Dec 4

epriestley added a comment to T784: Allow Differential changesets to be marked with various attributes.

Mostly, there's just relatively little interest from paying customers right now. PHI64, PHI675, PHI723 and PHI985 are all somewhat interested, but they're generally low-priority / "nice-to-have" sorts of requests, or this is a tangential or supplemental approach that might help with some other problem the request focuses on. Since there's limited customer interest, this isn't a very high priority right now.

Tue, Dec 4, 1:48 PM · Restricted Project, Restricted Project, Restricted Project, Arcanist, Differential
Pawka added a comment to T784: Allow Differential changesets to be marked with various attributes.

Is this functionality blocked on something else?

Tue, Dec 4, 12:57 PM · Restricted Project, Restricted Project, Restricted Project, Arcanist, Differential
Pawka added a project to T784: Allow Differential changesets to be marked with various attributes: Restricted Project.
Tue, Dec 4, 12:36 PM · Restricted Project, Restricted Project, Restricted Project, Arcanist, Differential

Mon, Nov 26

epriestley closed T12920: Retracting requested changes can leave revision in "Needs Revision" state as Resolved by committing rP97e7ef0f015c: When the last rejecting reviewer resigns from a revision, return it to "Needs….
Mon, Nov 26, 6:11 PM · Restricted Project, Differential, Bug Report
epriestley added a project to T13223: "Land Revision" builds a commit message as an omnipotent user, not the revision author or landing user: Drydock.
Mon, Nov 26, 5:53 PM · Drydock, Policy, Differential, Security
epriestley triaged T13223: "Land Revision" builds a commit message as an omnipotent user, not the revision author or landing user as Low priority.
Mon, Nov 26, 5:53 PM · Drydock, Policy, Differential, Security
epriestley added a revision to T12920: Retracting requested changes can leave revision in "Needs Revision" state: D19840: When the last rejecting reviewer resigns from a revision, return it to "Needs Review".
Mon, Nov 26, 2:06 PM · Restricted Project, Differential, Bug Report
epriestley updated the task description for T13161: Plans: Improve Differential diff display behaviors.
Mon, Nov 26, 1:01 PM · Differential, Plans

Sat, Nov 24

epriestley updated the task description for T13161: Plans: Improve Differential diff display behaviors.
Sat, Nov 24, 3:17 PM · Differential, Plans
epriestley updated the task description for T13161: Plans: Improve Differential diff display behaviors.
Sat, Nov 24, 3:11 PM · Differential, Plans
epriestley merged task T5032: In diff view, make the LHS text copy-able into T12822: Replace Differential ZWS "oncopy" behavior with a grand Javascript sorcery.
Sat, Nov 24, 3:09 PM · Differential
epriestley merged T5032: In diff view, make the LHS text copy-able into T12822: Replace Differential ZWS "oncopy" behavior with a grand Javascript sorcery.
Sat, Nov 24, 3:09 PM · Differential
epriestley updated the task description for T784: Allow Differential changesets to be marked with various attributes.
Sat, Nov 24, 3:08 PM · Restricted Project, Restricted Project, Restricted Project, Arcanist, Differential

Wed, Nov 21

epriestley renamed T12822: Replace Differential ZWS "oncopy" behavior with a grand Javascript sorcery from zero width space should be visible and emphasized in differential tool to Replace Differential ZWS "oncopy" behavior with a grand Javascript sorcery.
Wed, Nov 21, 5:28 PM · Differential

Fri, Nov 16

epriestley added a commit to T13054: Plans: Harbormaster Interactions with Differential / Chaos Reduction: rPb2e91d220539: Move the "container updated" message for Buildables that build Diffs outside of….
Fri, Nov 16, 8:34 PM · Plans, Differential, Harbormaster
epriestley added a commit to T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential: rP44c32839a68c: When you "Request Review" of a draft revision, change the button text from….
Fri, Nov 16, 4:50 AM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential

Thu, Nov 15

epriestley added a revision to T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential: D19810: When you "Request Review" of a draft revision, change the button text from "Submit Quietly" to "Publish Revision".
Thu, Nov 15, 1:14 PM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential
epriestley added a revision to T13054: Plans: Harbormaster Interactions with Differential / Chaos Reduction: D19807: Move the "container updated" message for Buildables that build Diffs outside of the transaction.
Thu, Nov 15, 12:10 AM · Plans, Differential, Harbormaster

Sep 24 2018

epriestley moved T13165: Copy detection in Git is very eager about empty files from Backlog to Far Future on the Arcanist board.
Sep 24 2018, 4:44 PM · Differential, Arcanist, Git
epriestley moved T1083: Update revisions with merge/rebase diffs so incremental changes can be shown from Backlog to Far Future on the Arcanist board.
Sep 24 2018, 3:48 PM · Differential, Arcanist
epriestley closed T3089: Using 'arc diff' after sending diffs through the web UI breaks the diff view. as Wontfix.

This is many years old, hasn't seen further reports, and no longer has usable reproduction instructions.

Sep 24 2018, 3:47 PM · Differential, Arcanist
epriestley closed T12010: Untangle the Gordian Knot of iterating on Differential/Diffusion/Arcanist as Resolved.

Piece by piece, this was effectively untangled. Differential largely moved forward; Arcanist has a path forward in T13098 (although it's ultimately a "major version" sort of change), and Diffusion stuff has settled into relatively actionable plans in, e.g., T13075.

Sep 24 2018, 3:46 PM · Diffusion (v3), Arcanist, Differential
epriestley added a parent task for T12010: Untangle the Gordian Knot of iterating on Differential/Diffusion/Arcanist: T13098: Plans: Arcanist toolsets and extensions.
Sep 24 2018, 3:36 PM · Diffusion (v3), Arcanist, Differential

Sep 22 2018

bcooksley added a comment to T13204: First mail sent by Differential doesn't have a Message-ID header set properly.

In my case (using GMail) it appears to thread properly as well however apparently some clients (KMail) rely on these headers more heavily than others.

Sep 22 2018, 12:07 AM · Mail, Differential

Sep 21 2018

epriestley added a comment to T13204: First mail sent by Differential doesn't have a Message-ID header set properly.

In order for threading to work in email clients, the Message-ID header needs to be set on the initial email, with following emails having In-Reply-To / References set to that same Message-ID.

Sep 21 2018, 11:25 PM · Mail, Differential
epriestley added a comment to T13204: First mail sent by Differential doesn't have a Message-ID header set properly.

I can't reproduce this: mail threads properly in my client.

Sep 21 2018, 11:23 PM · Mail, Differential
bcooksley created T13204: First mail sent by Differential doesn't have a Message-ID header set properly.
Sep 21 2018, 10:56 PM · Mail, Differential

Sep 17 2018

joshuaspence added a revision to T11699: `arc patch` is timing out: D19680: Don't pass `revisionIDs` to `differential.querydiffs`.
Sep 17 2018, 4:57 AM · Conduit, Differential

Sep 14 2018

epriestley updated the task description for T13161: Plans: Improve Differential diff display behaviors.
Sep 14 2018, 7:07 PM · Differential, Plans
epriestley updated the task description for T13161: Plans: Improve Differential diff display behaviors.
Sep 14 2018, 3:59 PM · Differential, Plans

Sep 12 2018

epriestley triaged T13200: "Responsible Users" queries with "Author" or "Reviewer" or "Auditor" constraints can include too many results as Low priority.
Sep 12 2018, 8:18 PM · Diffusion, Differential

Sep 10 2018

GoogleLegacy added a comment to T7794: Differential revision list "draft notification" bubble should ignore deleted drafts.
Sep 10 2018, 9:48 AM · Differential, Audit
GoogleLegacy added a comment to T13057: Formalize "reverts" and "resurrects" between revisions.
Sep 10 2018, 8:41 AM · Differential

Sep 7 2018

epriestley closed T8573: All reviewers should be allowed to mark a comment as done, a subtask of T1460: Add "Done" link to an inline comment, as Resolved.
Sep 7 2018, 6:18 PM · User Delight, Differential
epriestley closed T8573: All reviewers should be allowed to mark a comment as done as Resolved by committing rP2a03367a50c0: When authors add inlines to their own revisions/commits, mark them as "Done" by….
Sep 7 2018, 6:18 PM · Restricted Project, Differential, Restricted Project
epriestley added a commit to T8573: All reviewers should be allowed to mark a comment as done: rP16a6fc8341ea: Allow reviewers to mark their own inlines as "Done" before they submit them.
Sep 7 2018, 6:17 PM · Restricted Project, Differential, Restricted Project

Sep 6 2018

epriestley added a commit to T8573: All reviewers should be allowed to mark a comment as done: rP650e74933abb: Update some inline comment logic to use more modern "Viewer"-oriented….
Sep 6 2018, 2:57 PM · Restricted Project, Differential, Restricted Project

Sep 4 2018

epriestley added a comment to T8573: All reviewers should be allowed to mark a comment as done.
  • D19634 allows reviewers to mark their own inlines as "Done" before they submit them. This addresses the case where you want to leave a non-actionable inline (like "This is much-improved.") but feel like leaving an inline like this is imposing on the author because they "must" check it off.
  • D19635 marks the author's own inlines as "Done" by default. They may un-mark them if they prefer (e.g., actually want to leave themselves a TODO). This addresses the case where authors feel like they "must" check off every comment, so they either reply in two passes (one to submit; one to check off all their own replies) or avoid replying.
Sep 4 2018, 7:16 PM · Restricted Project, Differential, Restricted Project
epriestley added a revision to T8573: All reviewers should be allowed to mark a comment as done: D19635: When authors add inlines to their own revisions/commits, mark them as "Done" by default.
Sep 4 2018, 7:08 PM · Restricted Project, Differential, Restricted Project
epriestley added a revision to T8573: All reviewers should be allowed to mark a comment as done: D19634: Allow reviewers to mark their own inlines as "Done" before they submit them.
Sep 4 2018, 7:02 PM · Restricted Project, Differential, Restricted Project
epriestley added a revision to T8573: All reviewers should be allowed to mark a comment as done: D19633: Update some inline comment logic to use more modern "Viewer"-oriented calls/variables.
Sep 4 2018, 6:31 PM · Restricted Project, Differential, Restricted Project

Aug 27 2018

epriestley updated the task description for T13161: Plans: Improve Differential diff display behaviors.
Aug 27 2018, 5:23 PM · Differential, Plans

Aug 25 2018

mribau_a added a comment to T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential.

@bgamari Did you ever find a solution?

Aug 25 2018, 5:51 AM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential

Aug 24 2018

epriestley added a commit to T13176: Differential now has a "/D123/new/" link: rP5e4d9dfa9219: Condition "Changes Since Last Action" Differential link on "first broadcast"….
Aug 24 2018, 5:04 PM · Differential
epriestley added a revision to T13176: Differential now has a "/D123/new/" link: D19598: Condition "Changes Since Last Action" Differential link on "first broadcast", not "new object".
Aug 24 2018, 4:52 PM · Differential

Aug 14 2018

epriestley added a comment to T13176: Differential now has a "/D123/new/" link.

This is also showing up in initial email, which is silly and pointless.

Aug 14 2018, 9:40 PM · Differential

Aug 10 2018

epriestley closed T12534: Differential still appears in "responsible users" query after resigning as a reviewer as Resolved.

I believe this was fixed by D17558 and discussed in T11050.

Aug 10 2018, 8:00 PM · Bug Report (Needs Information), Differential

Jul 30 2018

epriestley triaged T13176: Differential now has a "/D123/new/" link as Wishlist priority.
Jul 30 2018, 3:03 PM · Differential

Jul 27 2018

epriestley closed T8163: Show changes since last accepted version as Resolved by committing rP9e451879d998: Add a "Changes Since Last Action" view to Differential revisions.
Jul 27 2018, 7:27 PM · Restricted Project, Differential
epriestley added a comment to T8163: Show changes since last accepted version.

I've marked D19541 as resolving this. The change there isn't quite what was asked for here, but overlaps significantly.

Jul 27 2018, 12:10 AM · Restricted Project, Differential