Page MenuHomePhabricator
Feed Advanced Search

Apr 30 2020

epriestley added a revision to T13513: Make schema changes to inlines to prepare for new inline features: Unknown Object (Differential Revision).
Apr 30 2020, 12:15 AM · Inline Comments, Diffusion, Differential

Apr 29 2020

epriestley added a revision to T13519: Support collapsing folders in the diff paths panel: Unknown Object (Differential Revision).
Apr 29 2020, 9:45 PM · Differential
epriestley added a comment to T13513: Make schema changes to inlines to prepare for new inline features.

As of D21188, inlines have these remaining weird behaviors:

Apr 29 2020, 7:16 PM · Inline Comments, Diffusion, Differential
epriestley added a revision to T13513: Make schema changes to inlines to prepare for new inline features: Unknown Object (Differential Revision).
Apr 29 2020, 7:00 PM · Inline Comments, Diffusion, Differential
epriestley added a revision to T13513: Make schema changes to inlines to prepare for new inline features: Unknown Object (Differential Revision).
Apr 29 2020, 6:34 PM · Inline Comments, Diffusion, Differential
epriestley added a revision to T13513: Make schema changes to inlines to prepare for new inline features: Unknown Object (Differential Revision).
Apr 29 2020, 2:56 PM · Inline Comments, Diffusion, Differential
epriestley added a comment to T13513: Make schema changes to inlines to prepare for new inline features.

In the DOM, inline comments are currently structured as a "comment row", followed by zero or more related rows (which may contain the editor UI or the undo UI).

Apr 29 2020, 2:12 PM · Inline Comments, Diffusion, Differential

Apr 28 2020

epriestley added a comment to T13513: Make schema changes to inlines to prepare for new inline features.

OKAY SO

Apr 28 2020, 9:29 PM · Inline Comments, Diffusion, Differential
epriestley added a revision to T13513: Make schema changes to inlines to prepare for new inline features: Unknown Object (Differential Revision).
Apr 28 2020, 8:48 PM · Inline Comments, Diffusion, Differential
epriestley added a revision to T13520: Update the diff table of contents for moved files, long paths, etc: Unknown Object (Differential Revision).
Apr 28 2020, 7:23 PM · Differential
epriestley added a revision to T13520: Update the diff table of contents for moved files, long paths, etc: Unknown Object (Differential Revision).
Apr 28 2020, 7:06 PM · Differential
epriestley moved T12982: In Differential, diff-of-diffs where both diffs added an identical file is misleading from Backlog to Intradiffs on the Differential board.
Apr 28 2020, 1:27 PM · Differential
epriestley moved T10500: When diffing B against A in Differential, files touched in A but not B should be finessed better from Backlog to Intradiffs on the Differential board.
Apr 28 2020, 1:26 PM · Restricted Project, Differential
epriestley moved T9792: Download raw diff from Differential weirdness regarding diff including binary files additions/deletions from Backlog to Intradiffs on the Differential board.
Apr 28 2020, 1:26 PM · Differential
epriestley moved T13523: Update logic around intradiff changesets to better handle binary files, header information, and attributes from Backlog to Intradiffs on the Differential board.
Apr 28 2020, 1:23 PM · Differential
epriestley renamed T13523: Update logic around intradiff changesets to better handle binary files, header information, and attributes from Update logic around interdiff changesets to better handle binary files, header information, and attributes to Update logic around intradiff changesets to better handle binary files, header information, and attributes.
Apr 28 2020, 1:22 PM · Differential
epriestley moved T1101: Allow Show Raw File for updated added files from v2 (UI/Mobile) to Intradiffs on the Differential board.
Apr 28 2020, 1:22 PM · Differential
epriestley added a revision to T13523: Update logic around intradiff changesets to better handle binary files, header information, and attributes: Unknown Object (Differential Revision).
Apr 28 2020, 12:00 PM · Differential
epriestley closed T13525: "apcu_store()" may raise "GC cache entry ..." errors as Resolved by committing Restricted Diffusion Commit.
Apr 28 2020, 11:13 AM · Infrastructure
epriestley added a comment to T13526: Error raised when "HarbormasterBuildPlanBehavior->getPlanOption()" is reachable with no plan.

I'm having trouble reproducing figuring out how to reproduce this locally.

Apr 28 2020, 11:05 AM · Harbormaster
epriestley triaged T13526: Error raised when "HarbormasterBuildPlanBehavior->getPlanOption()" is reachable with no plan as Low priority.
Apr 28 2020, 10:57 AM · Harbormaster
epriestley added a revision to T13525: "apcu_store()" may raise "GC cache entry ..." errors: Unknown Object (Differential Revision).
Apr 28 2020, 10:56 AM · Infrastructure
epriestley triaged T13525: "apcu_store()" may raise "GC cache entry ..." errors as Normal priority.
Apr 28 2020, 10:52 AM · Infrastructure

Apr 27 2020

epriestley added a comment to T13523: Update logic around intradiff changesets to better handle binary files, header information, and attributes.

These issues are somewhat complicated because changesets currently have a single changeType attribute and a single fileType attribute, and a lot of rendering logic depends on these attributes. This model isn't correct in the general case.

Apr 27 2020, 10:42 PM · Differential
epriestley closed T13524: "Undefined index" exception in ChangesetParser while viewing diffs as Resolved.

This seems to be fixed and has deployed to the cluster.

Apr 27 2020, 9:56 PM · Differential
epriestley added a revision to T13524: "Undefined index" exception in ChangesetParser while viewing diffs: Unknown Object (Differential Revision).
Apr 27 2020, 9:18 PM · Differential
epriestley added a comment to T13524: "Undefined index" exception in ChangesetParser while viewing diffs.

This appears to be an issue with synthetic lint messages from Harbormaster that do not specify a line number. We reach this section of the code:

Apr 27 2020, 9:09 PM · Differential
epriestley triaged T13524: "Undefined index" exception in ChangesetParser while viewing diffs as Normal priority.
Apr 27 2020, 8:51 PM · Differential
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 27 2020, 7:41 PM · Arcanist, Installing & Upgrading
epriestley edited the content of 2020 Week 16 (Late April).
Apr 27 2020, 7:15 PM
epriestley triaged T13523: Update logic around intradiff changesets to better handle binary files, header information, and attributes as Normal priority.
Apr 27 2020, 3:45 PM · Differential
epriestley added a revision to T13522: Compute an internal content hash of large files using hash-of-hashes: Unknown Object (Differential Revision).
Apr 27 2020, 3:29 PM · Differential, Files
epriestley triaged T13522: Compute an internal content hash of large files using hash-of-hashes as Wishlist priority.
Apr 27 2020, 2:48 PM · Differential, Files
epriestley updated the task description for T13455: Make "View Options" in Differential sticky across reloads.
Apr 27 2020, 2:32 PM · Differential
epriestley closed T13455: Make "View Options" in Differential sticky across reloads as Resolved.
  • Followups on "Hidden" logic have moved to T13519.
Apr 27 2020, 2:32 PM · Differential
epriestley closed T13455: Make "View Options" in Differential sticky across reloads, a subtask of T13515: Differential: Inlines and View Options, as Resolved.
Apr 27 2020, 2:32 PM · Differential
epriestley closed T13516: Remove the "filetree" setting (so it's always enabled for everyone), a subtask of T13515: Differential: Inlines and View Options, as Resolved.
Apr 27 2020, 2:31 PM · Differential
epriestley closed T13516: Remove the "filetree" setting (so it's always enabled for everyone) as Resolved.
  • Collapsing paths has moved forward to T13519.
  • Table of Contents updates have moved forward to T13520.
  • FormationView issues have moved forward to T13521.
Apr 27 2020, 2:31 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 27 2020, 2:30 PM · Diffusion, Differential
epriestley triaged T13521: FormationView could do more Javascript magic around device layouts and header elements as Low priority.
Apr 27 2020, 2:30 PM · Differential
epriestley triaged T13520: Update the diff table of contents for moved files, long paths, etc as Normal priority.
Apr 27 2020, 2:24 PM · Differential
epriestley triaged T13519: Support collapsing folders in the diff paths panel as Low priority.
Apr 27 2020, 2:15 PM · Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 27 2020, 2:10 PM · Diffusion, Differential

Apr 26 2020

epriestley added a revision to T13518: PHP 7.4 "preg_match_all()" + "PREG_OFFSET_CAPTURE" behavior has changed: Unknown Object (Differential Revision).
Apr 26 2020, 3:32 PM · Infrastructure
epriestley added a revision to T13518: PHP 7.4 "preg_match_all()" + "PREG_OFFSET_CAPTURE" behavior has changed: Unknown Object (Differential Revision).
Apr 26 2020, 3:30 PM · Infrastructure
epriestley updated the task description for T13518: PHP 7.4 "preg_match_all()" + "PREG_OFFSET_CAPTURE" behavior has changed.
Apr 26 2020, 3:27 PM · Infrastructure
epriestley triaged T13518: PHP 7.4 "preg_match_all()" + "PREG_OFFSET_CAPTURE" behavior has changed as Low priority.
Apr 26 2020, 3:27 PM · Infrastructure
epriestley added a comment to T13232: Mac OS X Mojave, SIP, Code Signing, and Apache.

With PHP 7.4, it seems OpenSSL (and various other libraries) have moved to pkg-config, which is a tool that uses the PKG_CONFIG_PATH environmental variable to locate packages. This impacts OpenSSL and libxml2.

Apr 26 2020, 3:02 PM · Infrastructure

Apr 25 2020

epriestley added a comment to T13493: JIRA API has changed identifiers from "key" to "accountId".

At time of writing, calls to rest/api/3/myself now return this:

Apr 25 2020, 9:04 PM · Auth
epriestley added a revision to T13493: JIRA API has changed identifiers from "key" to "accountId": Unknown Object (Differential Revision).
Apr 25 2020, 9:01 PM · Auth
epriestley added a comment to T13493: JIRA API has changed identifiers from "key" to "accountId".

At time of writing, calls to rest/auth/1/session return a result like this:

Apr 25 2020, 8:59 PM · Auth
epriestley closed T13517: Cloning repositories over HTTPS may fail after compression changes as Resolved.

I believe D21167 fixed this.

Apr 25 2020, 4:24 PM · Diffusion
epriestley added a revision to T13517: Cloning repositories over HTTPS may fail after compression changes: Unknown Object (Differential Revision).
Apr 25 2020, 2:40 PM · Diffusion
epriestley added a comment to T13517: Cloning repositories over HTTPS may fail after compression changes.

The request sequence here is:

Apr 25 2020, 2:37 PM · Diffusion
epriestley triaged T13517: Cloning repositories over HTTPS may fail after compression changes as Normal priority.
Apr 25 2020, 2:29 PM · Diffusion

Apr 24 2020

epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 24 2020, 6:21 PM · Diffusion, Differential
epriestley added a revision to T4369: Phabricator HTTP repository hosting has fairly severe scalability limits: Unknown Object (Differential Revision).
Apr 24 2020, 6:10 PM · Diffusion (v3), Restricted Project
epriestley edited the content of Changelog.
Apr 24 2020, 3:48 PM
epriestley created an object: 2020 Week 16 (Late April).
Apr 24 2020, 3:47 PM
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 24 2020, 12:20 PM · Diffusion, Differential
epriestley added a comment to T13493: JIRA API has changed identifiers from "key" to "accountId".

See private correspondence ("Re: Contributing / Jira Oauth Patch"), which suggests the call to rest/auth/1/session should be (and may urgently need to be) replaced with a call to rest/api/2/myself. See also https://developer.atlassian.com/cloud/jira/platform/deprecation-notice-basic-auth-and-cookie-based-auth/#which-apis-and-methods-will-be-restricted-.

Apr 24 2020, 4:45 AM · Auth

Apr 23 2020

epriestley updated the task description for T13455: Make "View Options" in Differential sticky across reloads.
Apr 23 2020, 9:52 PM · Differential
epriestley added a revision to T13455: Make "View Options" in Differential sticky across reloads: Unknown Object (Differential Revision).
Apr 23 2020, 9:01 PM · Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 23 2020, 8:34 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 23 2020, 8:34 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 23 2020, 8:17 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 23 2020, 8:05 PM · Diffusion, Differential
epriestley updated the task description for T13455: Make "View Options" in Differential sticky across reloads.
Apr 23 2020, 5:16 PM · Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 23 2020, 5:10 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 23 2020, 5:09 PM · Diffusion, Differential
epriestley updated the task description for T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions.
Apr 23 2020, 5:08 PM · Plans, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 23 2020, 5:07 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 23 2020, 3:11 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 23 2020, 1:48 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 23 2020, 1:47 PM · Diffusion, Differential

Apr 22 2020

epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 22 2020, 8:51 PM · Diffusion, Differential
epriestley closed T7467: Consider giving deleted files a more prominent treatment in the changeset UI as Resolved.

This is quite old and I'm not sure it's particularly relevant anymore. T13516 has made some related changes.

Apr 22 2020, 8:51 PM · Restricted Project, Differential
epriestley updated the task description for T13515: Differential: Inlines and View Options.
Apr 22 2020, 8:50 PM · Differential
epriestley updated the task description for T13513: Make schema changes to inlines to prepare for new inline features.
Apr 22 2020, 8:49 PM · Inline Comments, Diffusion, Differential
epriestley added a revision to T13455: Make "View Options" in Differential sticky across reloads: Unknown Object (Differential Revision).
Apr 22 2020, 8:24 PM · Differential
epriestley updated the task description for T13455: Make "View Options" in Differential sticky across reloads.
Apr 22 2020, 8:24 PM · Differential
epriestley added a comment to T13455: Make "View Options" in Differential sticky across reloads.

Currently, I also think no part of this pipeline can actually tell if changeset X changed at diff Y or not, but this isn't conceptually hard to get right most of the time.

Apr 22 2020, 7:14 PM · Differential
epriestley closed T7469: Handle folder copies/moves more gracefully as Wontfix.

I'm not aware of any current customer interest in improving this behavior, and it's likely a lot of work to improve what is largely a moot case (there's usually no value in reviewing individual file-level changes in a "create a new branch X" commit) that only affects Subversion.

Apr 22 2020, 6:19 PM · Diffusion, Subversion
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 22 2020, 6:00 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 22 2020, 3:53 PM · Diffusion, Differential
epriestley added a comment to T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions.

Filetree stuff is now in T13516: Remove the "filetree" setting (so it's always enabled for everyone).

Apr 22 2020, 3:21 PM · Plans, Differential
epriestley updated the task description for T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions.
Apr 22 2020, 3:20 PM · Plans, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 22 2020, 3:20 PM · Diffusion, Differential
nfirmani awarded T13516: Remove the "filetree" setting (so it's always enabled for everyone) a 100 token.
Apr 22 2020, 2:14 AM · Diffusion, Differential

Apr 21 2020

epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 21 2020, 10:56 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 21 2020, 10:21 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 21 2020, 9:51 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 21 2020, 9:04 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 21 2020, 9:03 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 21 2020, 8:36 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 21 2020, 6:54 PM · Diffusion, Differential
epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 21 2020, 6:27 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Unknown Object (Differential Revision).
Apr 21 2020, 4:45 PM · Diffusion, Differential
epriestley added a revision to T13516: Remove the "filetree" setting (so it's always enabled for everyone): Restricted Differential Revision.
Apr 21 2020, 2:23 PM · Diffusion, Differential

Apr 20 2020

epriestley updated the task description for T13516: Remove the "filetree" setting (so it's always enabled for everyone).
Apr 20 2020, 10:13 PM · Diffusion, Differential