Page MenuHomePhabricator
Feed All Stories

Mon, Mar 22

epriestley added a comment to T13534: Inline Comment errata after editing persistence.

When you are viewing a change under engine X, and comments made under engine Y are present, they are not handled specially.

Mon, Mar 22, 5:32 PM · Differential
epriestley updated the task description for T13642: Inline comment line numbers on DocumentEngine block diffs may be clamped to raw lines in the source file.
Mon, Mar 22, 5:31 PM · Differential

Sat, Mar 20

epriestley added a comment to D21624: Remove or correct various "phabricator/" references to "libphutil".

Diviner updates based on the deployed software version on secure, and I just haven't deployed this yet. Stuff usually gets deployed in about a day, this has been on the slower side since I've been moving furniture.

Sat, Mar 20, 3:55 PM
jrtc27 added a comment to D21624: Remove or correct various "phabricator/" references to "libphutil".

I stumbled across its mention in the Arcanist Quick Start whilst setting up a new machine today so went to write a patch before realising that you'd just fixed this a few days ago. Is it expected that Diviner has yet to update?

Sat, Mar 20, 10:58 AM

Thu, Mar 18

epriestley updated the task description for T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting.
Thu, Mar 18, 6:45 PM · Accessibility, Differential
epriestley requested review of D21642: Fix an issue where inlines could be duplicated in the client list.
Thu, Mar 18, 6:40 PM
epriestley updated the task description for T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting.
Thu, Mar 18, 6:39 PM · Accessibility, Differential
epriestley added a revision to T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting: D21642: Fix an issue where inlines could be duplicated in the client list.
Thu, Mar 18, 6:33 PM · Accessibility, Differential

Wed, Mar 17

epriestley added a comment to T13290: Clean up remaining "Autoclose" behaviors related to "One Revision, Many Commits".

A general concern with "batch processing" is that it's quite bad if one commit failing to import can stall the entire repository forever.

Wed, Mar 17, 11:18 PM · Diffusion, Differential
epriestley added a comment to T13290: Clean up remaining "Autoclose" behaviors related to "One Revision, Many Commits".

See also T13552, which modifies the above discussion. The "Update" steps now happen after the "Publish" step.

Wed, Mar 17, 11:12 PM · Diffusion, Differential
epriestley closed T13425: Support diffs between abstract block lists in the UI as Resolved.

The bulk of this work is done and I think there's nothing unique and actionable left here. This is survived by T13642 and other issues.

Wed, Mar 17, 11:07 PM · Differential
epriestley closed T13515: Differential: Inlines and View Options as Resolved.

Survived by T13534, etc.

Wed, Mar 17, 11:01 PM · Differential
epriestley updated the task description for T13534: Inline Comment errata after editing persistence.
Wed, Mar 17, 11:00 PM · Differential
epriestley updated the task description for T13515: Differential: Inlines and View Options.
Wed, Mar 17, 11:00 PM · Differential
epriestley closed T13513: Make schema changes to inlines to prepare for new inline features as Resolved.

When you are viewing a document with engine X, and comments originally made with engine Y are present, this should be indicated ("This comment was made while looking at this change as a Q document."). They should probably also be moved to the top/bottom of the file, at least by default, since "Jupyter line number 9 = raw source line number 9" is an exceptionally bad and confusing guess at how to map line numbers.

Wed, Mar 17, 10:59 PM · Inline Comments, Diffusion, Differential
epriestley closed T13513: Make schema changes to inlines to prepare for new inline features, a subtask of T13515: Differential: Inlines and View Options, as Resolved.
Wed, Mar 17, 10:59 PM · Differential
epriestley updated the task description for T13534: Inline Comment errata after editing persistence.
Wed, Mar 17, 10:59 PM · Differential
epriestley closed T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch as Resolved.

Some tag stuff ended up here; I moved it to T13645.

Wed, Mar 17, 10:47 PM · Differential
epriestley updated the task description for T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch.
Wed, Mar 17, 10:46 PM · Differential
epriestley triaged T13645: In Diffusion "Tags" view, improve behavior for tags that point at tag objects as Low priority.
Wed, Mar 17, 10:45 PM · Diffusion
epriestley closed T13638: Investigate "AllowEncodedSlashes" Apache setting as Resolved.

Given the generally high degree of mootness here, I'm not planning to add a setup test or recommend installs adjust the setting.

Wed, Mar 17, 10:40 PM · Setup
epriestley closed T13644: Command-clicking a changeset header in Differential attempts to perform symbol lookup as Resolved.
Wed, Mar 17, 10:32 PM · Differential
epriestley committed rPdb9191f9a8d5: Correct minor "jump to symbol" behavior in Differential (authored by epriestley).
Correct minor "jump to symbol" behavior in Differential
Wed, Mar 17, 10:32 PM
epriestley closed D21641: Correct minor "jump to symbol" behavior in Differential.
Wed, Mar 17, 10:32 PM
epriestley requested review of D21641: Correct minor "jump to symbol" behavior in Differential.
Wed, Mar 17, 10:31 PM
epriestley added a revision to T13644: Command-clicking a changeset header in Differential attempts to perform symbol lookup: D21641: Correct minor "jump to symbol" behavior in Differential.
Wed, Mar 17, 10:30 PM · Differential
epriestley added a revision to T13638: Investigate "AllowEncodedSlashes" Apache setting: D21641: Correct minor "jump to symbol" behavior in Differential.
Wed, Mar 17, 10:30 PM · Setup
epriestley added a comment to T13638: Investigate "AllowEncodedSlashes" Apache setting.

Previously, see T13437.

Wed, Mar 17, 10:15 PM · Setup
epriestley added a comment to T13638: Investigate "AllowEncodedSlashes" Apache setting.

This reproduces locally.

Wed, Mar 17, 10:11 PM · Setup
epriestley added a comment to T13638: Investigate "AllowEncodedSlashes" Apache setting.

See also T13644, which appears to be the origin of the particular requests observed in PHI2021.

Wed, Mar 17, 9:38 PM · Setup
epriestley committed rP527dd3ce50b2: Replace Differential "unit stars" with icons (authored by epriestley).
Replace Differential "unit stars" with icons
Wed, Mar 17, 8:46 PM
epriestley committed rPd6ed9392d4e0: Replace Differential "lint stars" with icons (authored by epriestley).
Replace Differential "lint stars" with icons
Wed, Mar 17, 8:46 PM
epriestley closed D21639: Replace Differential "unit stars" with icons.
Wed, Mar 17, 8:46 PM
epriestley closed D21640: Replace Differential "lint stars" with icons.
Wed, Mar 17, 8:46 PM
epriestley closed T9764: Lint star colors have different meaning for revisions and diffs as Resolved.
  • Colors are now consistent.
  • Icons no longer use only color to communicate information.
  • Icons are (mostly) consistent with the Harbormaster icons that have similar meanings.
  • See ConfigModules/ExtensionsConstants: Differential.
Wed, Mar 17, 8:45 PM · Restricted Project, Differential
epriestley requested review of D21640: Replace Differential "lint stars" with icons.
Wed, Mar 17, 8:45 PM
epriestley added a revision to T9764: Lint star colors have different meaning for revisions and diffs: D21640: Replace Differential "lint stars" with icons.
Wed, Mar 17, 8:44 PM · Restricted Project, Differential
epriestley requested review of D21639: Replace Differential "unit stars" with icons.
Wed, Mar 17, 8:28 PM
epriestley added a revision to T9764: Lint star colors have different meaning for revisions and diffs: D21639: Replace Differential "unit stars" with icons.
Wed, Mar 17, 8:26 PM · Restricted Project, Differential
epriestley closed T8591: Undefined index in changeset parser as Resolved.

The diff above no longer reproduces this, and I think an equivalent to D18302 landed in D21178. T13524 was almost certainly the same issue.

Wed, Mar 17, 7:32 PM · Differential
epriestley abandoned D18302: Fix "undefined index" error in Differential changeset parser.

I think this was mooted by the similar change in D21178. See T8591 for followup.

Wed, Mar 17, 7:31 PM
epriestley commandeered D18302: Fix "undefined index" error in Differential changeset parser.
Wed, Mar 17, 7:31 PM
epriestley moved T13644: Command-clicking a changeset header in Differential attempts to perform symbol lookup from Backlog to Next on the Differential board.
Wed, Mar 17, 7:30 PM · Differential
epriestley triaged T13644: Command-clicking a changeset header in Differential attempts to perform symbol lookup as Low priority.
Wed, Mar 17, 7:29 PM · Differential
epriestley closed T9298: Lint/unittest excuses overwrite each other as Resolved.

This is mooted because I've removed the "excuses" feature about a year ago -- on the balance, I think it generated more confusion and busywork than signal.

Wed, Mar 17, 7:26 PM · Differential
epriestley renamed T13643: Permissions and UI around manual web "Update Diff" workflow are inconsistent from Permissions around manual web "Update Diff" workflow are inconsistent to Permissions and UI around manual web "Update Diff" workflow are inconsistent.
Wed, Mar 17, 7:23 PM · Differential
epriestley closed T9499: Revision repository is not updated when it is changed in the UI as Resolved.

The modern differential.revision.search API returns the repository for the revision.

Wed, Mar 17, 7:23 PM · Differential, Conduit
epriestley committed rP1d1003af7854: When using "Update Diff" from the web UI, prefill "Repository" properly (authored by epriestley).
When using "Update Diff" from the web UI, prefill "Repository" properly
Wed, Mar 17, 7:23 PM
epriestley closed D21638: When using "Update Diff" from the web UI, prefill "Repository" properly.
Wed, Mar 17, 7:23 PM
epriestley moved T13643: Permissions and UI around manual web "Update Diff" workflow are inconsistent from Backlog to Future Work on the Differential board.
Wed, Mar 17, 7:22 PM · Differential
epriestley triaged T13643: Permissions and UI around manual web "Update Diff" workflow are inconsistent as Wishlist priority.
Wed, Mar 17, 7:22 PM · Differential
epriestley requested review of D21638: When using "Update Diff" from the web UI, prefill "Repository" properly.
Wed, Mar 17, 7:20 PM
epriestley added a revision to T9499: Revision repository is not updated when it is changed in the UI: D21638: When using "Update Diff" from the web UI, prefill "Repository" properly.
Wed, Mar 17, 7:19 PM · Differential, Conduit
epriestley moved T12256: Parse multiple commits and commit metadata from "git show" and "git format-patch" from Backlog to Diff Parsing on the Differential board.
Wed, Mar 17, 7:07 PM · Differential, KDE
epriestley moved T12618: Parse multiple commits and commit metadata from "hg log --patch" and "hg export" from Backlog to Diff Parsing on the Differential board.
Wed, Mar 17, 7:07 PM · Arcanist, Mercurial, Differential, Conduit, Feature Request
epriestley moved T12664: Update diff/patch parsing to extract more metadata and parse a wider range of formats from Backlog to Diff Parsing on the Differential board.
Wed, Mar 17, 7:07 PM · Arcanist, Differential
epriestley moved T13432: Diffs that have had trailing whitespace trimmed or were generated with the Git "diff.suppressBlankEmpty" config option render oddly from Backlog to Diff Parsing on the Differential board.
Wed, Mar 17, 7:07 PM · Differential
epriestley added a comment to T9764: Lint star colors have different meaning for revisions and diffs.

See also PHI1910.

Wed, Mar 17, 7:06 PM · Restricted Project, Differential
epriestley moved T10219: Show suggested lint autofix in Differential web view from Backlog to Next on the Differential board.
Wed, Mar 17, 7:04 PM · Lint, Differential
epriestley moved T9947: "Expand File" keyboard shortcut in Differential should work on files which were not loaded by default from Backlog to Next on the Differential board.
Wed, Mar 17, 7:04 PM · Restricted Project, Differential, Feature Request
epriestley moved T9764: Lint star colors have different meaning for revisions and diffs from Backlog to Next on the Differential board.
Wed, Mar 17, 7:04 PM · Restricted Project, Differential
epriestley moved T9590: Subversion directory anchoring rules are unclear and underpowered from Backlog to Subversion on the Differential board.
Wed, Mar 17, 7:03 PM · Differential, Subversion
epriestley moved T9499: Revision repository is not updated when it is changed in the UI from Backlog to Next on the Differential board.
Wed, Mar 17, 7:03 PM · Differential, Conduit
epriestley moved T9298: Lint/unittest excuses overwrite each other from Backlog to Next on the Differential board.
Wed, Mar 17, 7:03 PM · Differential
epriestley moved T8591: Undefined index in changeset parser from Backlog to Next on the Differential board.
Wed, Mar 17, 7:03 PM · Differential
epriestley moved T4453: When multiple commits correspond to a single revision, Differential updates to show one of them arbitrarily from Next to Backlog on the Differential board.
Wed, Mar 17, 7:01 PM · Restricted Project, Prioritized, Differential
epriestley moved T4453: When multiple commits correspond to a single revision, Differential updates to show one of them arbitrarily from Backlog to Next on the Differential board.
Wed, Mar 17, 7:01 PM · Restricted Project, Prioritized, Differential
epriestley moved T4305: File state not saved in revision when parsing form svn repo from Backlog to Subversion on the Differential board.
Wed, Mar 17, 7:01 PM · Subversion, Differential
epriestley closed T7677: 1-up Renderer should probably render comments full-width as Wontfix.

I think the complexity of changing this probably isn't worth the benefit. The complexity is high because of how much client-side interaction there is around inlines, and even with more space for the inline, the example screenshot doesn't seem great for code review anyway since none of the code fits.

Wed, Mar 17, 7:01 PM · Restricted Project, Differential
epriestley moved T4678: Support changelist-based SVN workflows in Arcanist and Differential from Backlog to Subversion on the Differential board.
Wed, Mar 17, 6:55 PM · Differential, Arcanist, Subversion
epriestley moved T4698: Support for SVN patches, created by PhpStorm from Backlog to Subversion on the Differential board.
Wed, Mar 17, 6:55 PM · Subversion, Differential
epriestley moved T5999: `arc diff` not uploading file content for svn:executable files under SVN 1.7.4 from Backlog to Subversion on the Differential board.
Wed, Mar 17, 6:54 PM · Arcanist, Differential, Subversion
epriestley moved T386: Differential display of revisions with "svn cp x@12345 x" is inaccurate from Backlog to Subversion on the Differential board.
Wed, Mar 17, 6:54 PM · Differential, Subversion
epriestley committed rPff0a4a2c6fce: Use a less-misleading example for Conduit custom constraints (authored by epriestley).
Use a less-misleading example for Conduit custom constraints
Wed, Mar 17, 6:47 PM
epriestley closed D21637: Use a less-misleading example for Conduit custom constraints.
Wed, Mar 17, 6:47 PM
epriestley requested review of D21637: Use a less-misleading example for Conduit custom constraints.
Wed, Mar 17, 6:32 PM
epriestley moved T13642: Inline comment line numbers on DocumentEngine block diffs may be clamped to raw lines in the source file from Backlog to Next on the Differential board.
Wed, Mar 17, 5:14 PM · Differential
epriestley added a comment to T13642: Inline comment line numbers on DocumentEngine block diffs may be clamped to raw lines in the source file.

In D21435, inlines were clamped. Adjusting this code is necessary, but not sufficient, to correct this problem.

Wed, Mar 17, 5:14 PM · Differential
epriestley triaged T13642: Inline comment line numbers on DocumentEngine block diffs may be clamped to raw lines in the source file as Low priority.
Wed, Mar 17, 4:57 PM · Differential

Tue, Mar 16

Harbormaster failed to build B25299: rP7c0e33c34d6f: Modernize "mailKey" for Almanac Bindings for rP7c0e33c34d6f: Modernize "mailKey" for Almanac Bindings!
Tue, Mar 16, 10:59 PM
Harbormaster failed to build B25295: rPc3e6db6f0b32: Migrate Almanac Device "mailKey" to modern storage for rPc3e6db6f0b32: Migrate Almanac Device "mailKey" to modern storage!
Tue, Mar 16, 10:56 PM
Harbormaster failed to build B25294: rP9003a453691f: Make minor Almanac device modernization updates for rP9003a453691f: Make minor Almanac device modernization updates!
Tue, Mar 16, 10:56 PM
epriestley committed rP86669ab54f97: Modernize "mailKey" for Almanac Networks (authored by epriestley).
Modernize "mailKey" for Almanac Networks
Tue, Mar 16, 10:52 PM
epriestley committed rP12341e4bc8b4: Forbid disabled devices from authenticating via SSH or HTTP (authored by epriestley).
Forbid disabled devices from authenticating via SSH or HTTP
Tue, Mar 16, 10:52 PM
epriestley closed D21632: Modernize "mailKey" for Almanac Networks.
Tue, Mar 16, 10:52 PM
epriestley committed rP3267859aeeb2: Modernize "mailKey" on Fund initiatives (authored by epriestley).
Modernize "mailKey" on Fund initiatives
Tue, Mar 16, 10:52 PM
epriestley committed rP7c0e33c34d6f: Modernize "mailKey" for Almanac Bindings (authored by epriestley).
Modernize "mailKey" for Almanac Bindings
Tue, Mar 16, 10:52 PM
epriestley committed rPe6a23274fee4: Default the Almanac Devices query to "Active Devices" (authored by epriestley).
Default the Almanac Devices query to "Active Devices"
Tue, Mar 16, 10:52 PM
epriestley closed D21635: Forbid disabled devices from authenticating via SSH or HTTP.
Tue, Mar 16, 10:52 PM
epriestley committed rPeeb009b4fa61: Modernize "mailKey" for Calendar Event (authored by epriestley).
Modernize "mailKey" for Calendar Event
Tue, Mar 16, 10:52 PM
epriestley closed D21631: Modernize "mailKey" on Almanac Namespaces.
Tue, Mar 16, 10:52 PM
epriestley closed D21633: Modernize "mailKey" for Calendar Event.
Tue, Mar 16, 10:52 PM
epriestley committed rP8226b3c88082: Modernize "mailKey" on Almanac Namespaces (authored by epriestley).
Modernize "mailKey" on Almanac Namespaces
Tue, Mar 16, 10:52 PM
epriestley closed D21634: Modernize "mailKey" on Fund initiatives.
Tue, Mar 16, 10:52 PM
epriestley committed rPfadb2bd52b5e: Modernize "mailKey" on AlamnacService (authored by epriestley).
Modernize "mailKey" on AlamnacService
Tue, Mar 16, 10:52 PM
epriestley closed D21636: Default the Almanac Devices query to "Active Devices".
Tue, Mar 16, 10:52 PM
epriestley closed D21630: Modernize "mailKey" for Almanac Bindings.
Tue, Mar 16, 10:52 PM
epriestley closed D21629: Modernize "mailKey" on AlamnacService.
Tue, Mar 16, 10:52 PM
epriestley committed rP5d64fb1815f7: Add a "status" property to Almanac devices (authored by epriestley).
Add a "status" property to Almanac devices
Tue, Mar 16, 10:52 PM
epriestley closed D21627: Add a "status" property to Almanac devices.
Tue, Mar 16, 10:52 PM