Page MenuHomePhabricator
Feed All Stories

Aug 12 2020

epriestley committed rP7fd6bf26a949: Modernize "Author" and "Committer" rendering for commits (authored by epriestley).
Modernize "Author" and "Committer" rendering for commits
Aug 12 2020, 3:58 PM
epriestley closed D21405: Modernize "Author" and "Committer" rendering for commits.
Aug 12 2020, 3:58 PM
epriestley committed rP81e4e5b7f9c9: Remove construction of "author" information from "LastModified" payload in… (authored by epriestley).
Remove construction of "author" information from "LastModified" payload in…
Aug 12 2020, 3:58 PM
epriestley closed D21404: Remove construction of "author" information from "LastModified" payload in Diffusion.
Aug 12 2020, 3:58 PM
epriestley requested review of D21440: Fix some content/background overflow issues with commit graph lists.
Aug 12 2020, 3:57 PM
epriestley added a revision to 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: D21440: Fix some content/background overflow issues with commit graph lists.
Aug 12 2020, 3:54 PM · Differential
arichardson abandoned D21438: Find the per-worktree .git directory when using git-worktree.
Aug 12 2020, 2:48 PM
epriestley committed rP545417597322: Coerce Chrome into breaking monospaced text when printing tables to PDFs (authored by epriestley).
Coerce Chrome into breaking monospaced text when printing tables to PDFs
Aug 12 2020, 2:35 PM
epriestley closed D21439: Coerce Chrome into breaking monospaced text when printing tables to PDFs.
Aug 12 2020, 2:35 PM
epriestley added a comment to D21438: Find the per-worktree .git directory when using git-worktree.

To be completely clear: I'm not going to accept any features upstream which zero customers are interested in. It doesn't matter if they're simple or complicated. If no customers want it, it isn't coming upstream.

Aug 12 2020, 2:17 PM
arichardson added a comment to D21438: Find the per-worktree .git directory when using git-worktree.

I don't expect any sophisticated git-worktree support. This is a simple change that allows arc get-config/set-config to work correctly when invoked from a worktree. It might also fix some other cases where arc expects $identity->localMetaDir to be a directory which previously wasn't the case for worktrees.

Aug 12 2020, 9:48 AM

Aug 11 2020

epriestley triaged T13567: Arcanist does not decode unicode filenames in Git before passing them to linters as Low priority.
Aug 11 2020, 6:38 PM · Arcanist
epriestley renamed T13566: Improve fallback behavior for "arc branches/bookmarks" when unpublished local state appears to have >1K commits from Improve fallback behavior for "arc branches/bookmarks" to Improve fallback behavior for "arc branches/bookmarks" when unpublished local state appears to have >1K commits.
Aug 11 2020, 6:34 PM · Arcanist
epriestley triaged T13566: Improve fallback behavior for "arc branches/bookmarks" when unpublished local state appears to have >1K commits as Normal priority.
Aug 11 2020, 6:34 PM · Arcanist
epriestley triaged T13565: Filesystem write failure error messages lack tailoring, notably in DeferredLog as Normal priority.
Aug 11 2020, 6:04 PM · Infrastructure
epriestley requested review of D21439: Coerce Chrome into breaking monospaced text when printing tables to PDFs.
Aug 11 2020, 5:52 PM
epriestley updated the task description for T13562: Windows may have irregular behaviors in unusual cases when copying a file to preserve attributes/permissions.
Aug 11 2020, 5:47 PM · Windows, Arcanist
epriestley added a comment to T13564: Remarkup tables should show all content when printed to PDF.

Adding overflow-wrap: anywhere appears to fix this without breaking anything.

Aug 11 2020, 5:35 PM · Google Chrome, Remarkup
epriestley triaged T13564: Remarkup tables should show all content when printed to PDF as Normal priority.
Aug 11 2020, 5:17 PM · Google Chrome, Remarkup
epriestley triaged T13563: On Windows, Arcanist tries and fails to set stdin nonblocking while reading the stream as Low priority.
Aug 11 2020, 5:06 PM · Windows, Arcanist
epriestley triaged T13562: Windows may have irregular behaviors in unusual cases when copying a file to preserve attributes/permissions as Normal priority.
Aug 11 2020, 5:02 PM · Windows, Arcanist
epriestley added a revision to T9924: Add support for git worktree: D21438: Find the per-worktree .git directory when using git-worktree.
Aug 11 2020, 4:45 PM · FreeBSD, Restricted Project, Arcanist, Feature Request
epriestley added a task to D21438: Find the per-worktree .git directory when using git-worktree: T9924: Add support for git worktree.
Aug 11 2020, 4:45 PM
epriestley added a comment to D21438: Find the per-worktree .git directory when using git-worktree.

There's currently no customer interest in worktree support. The use cases in T9924 seem flimsy to me so I don't currently plan to support worktrees in the absence of customer interest. See also ArcanistGitWorkingCopy.

Aug 11 2020, 4:44 PM
arichardson updated the diff for D21438: Find the per-worktree .git directory when using git-worktree.
  • fix $vcs -> $vcs_type
Aug 11 2020, 3:54 PM
arichardson requested review of D21438: Find the per-worktree .git directory when using git-worktree.
Aug 11 2020, 3:52 PM
epriestley updated the task description for T13561: Allow the daemons to generate tasks dynamically at runtime.
Aug 11 2020, 3:26 PM · Daemons
epriestley triaged T13561: Allow the daemons to generate tasks dynamically at runtime as Normal priority.
Aug 11 2020, 3:15 PM · Daemons
epriestley added a comment to T13125: Plans: Code coverage reporting.

PHI1816 would also like dimensional coverage: coverage by language; coverage by team/owner. This is perhaps worth considering in how coverage is aggregated.

Aug 11 2020, 2:54 PM · Diffusion, Plans
epriestley triaged T11372: Multiple coverage columns for different types of code coverage as Wishlist priority.
Aug 11 2020, 2:49 PM · Diffusion, Restricted Project
epriestley updated the task description for T13125: Plans: Code coverage reporting.
Aug 11 2020, 2:48 PM · Diffusion, Plans
epriestley added subtasks for T13125: Plans: Code coverage reporting: T11372: Multiple coverage columns for different types of code coverage, T9524: Show aggregate subdirectory coverage in Diffusion browse views.
Aug 11 2020, 2:47 PM · Diffusion, Plans
epriestley added a parent task for T9524: Show aggregate subdirectory coverage in Diffusion browse views: T13125: Plans: Code coverage reporting.
Aug 11 2020, 2:47 PM · Diffusion, Restricted Project
epriestley added a parent task for T11372: Multiple coverage columns for different types of code coverage: T13125: Plans: Code coverage reporting.
Aug 11 2020, 2:47 PM · Diffusion, Restricted Project

Aug 10 2020

epriestley committed rP79375c6c5398: Make "Quote" work properly in Pholio (authored by epriestley).
Make "Quote" work properly in Pholio
Aug 10 2020, 8:40 PM
epriestley closed D21437: Make "Quote" work properly in Pholio.
Aug 10 2020, 8:40 PM
epriestley triaged T13560: "Large number of changes" transaction timeline banner may hide a single change as Wishlist priority.
Aug 10 2020, 8:38 PM · Design
epriestley requested review of D21437: Make "Quote" work properly in Pholio.
Aug 10 2020, 8:33 PM
epriestley added a project to T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting: Accessibility.
Aug 10 2020, 8:18 PM · Accessibility, Differential
epriestley triaged T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting as Wishlist priority.
Aug 10 2020, 8:18 PM · Accessibility, Differential
epriestley triaged T13558: Duration computation on user profiles is silly as Wishlist priority.
Aug 10 2020, 8:15 PM · Design

Aug 8 2020

epriestley edited the content of Changelog.
Aug 8 2020, 2:39 AM
epriestley committed rP5e3768473711: (stable) Promote 2020 Week 31 (authored by epriestley).
(stable) Promote 2020 Week 31
Aug 8 2020, 2:38 AM
epriestley created an object: 2020 Week 31 (Early August).
Aug 8 2020, 2:38 AM

Aug 5 2020

epriestley committed rPce0dc9a2ba2c: Correct an apparent off-by-one error when adjusting inlines across revision… (authored by epriestley).
Correct an apparent off-by-one error when adjusting inlines across revision…
Aug 5 2020, 8:13 PM
epriestley committed rPdbdfac1e072e: Recover inline comments which are "adjusted" off the end of a diff (authored by epriestley).
Recover inline comments which are "adjusted" off the end of a diff
Aug 5 2020, 8:13 PM
epriestley closed D21436: Correct an apparent off-by-one error when adjusting inlines across revision changes.
Aug 5 2020, 8:12 PM
epriestley closed D21435: Recover inline comments which are "adjusted" off the end of a diff.
Aug 5 2020, 8:12 PM
epriestley requested review of D21436: Correct an apparent off-by-one error when adjusting inlines across revision changes.
Aug 5 2020, 8:11 PM
epriestley requested review of D21435: Recover inline comments which are "adjusted" off the end of a diff.
Aug 5 2020, 8:01 PM
epriestley committed rP2db1955159f7: In Jupyter notebooks, read strings stored in the raw as either "string" or… (authored by epriestley).
In Jupyter notebooks, read strings stored in the raw as either "string" or…
Aug 5 2020, 7:26 PM
epriestley closed D21434: In Jupyter notebooks, read strings stored in the raw as either "string" or "list<string>" more consistently.
Aug 5 2020, 7:26 PM
epriestley updated the task description for T13241: Guide: SMS is Insecure.
Aug 5 2020, 7:22 PM · Security, Guides
epriestley requested review of D21434: In Jupyter notebooks, read strings stored in the raw as either "string" or "list<string>" more consistently.
Aug 5 2020, 7:20 PM
epriestley added a comment to D21225: Make the "attach_inlines" parameter to "differential.createcomment" a no-op.

With newer inline comment functionality there is no ability to create submitted inline comments for bots or via API directly.

Aug 5 2020, 6:59 PM
artms added a comment to D21225: Make the "attach_inlines" parameter to "differential.createcomment" a no-op.

I'm not aware of any current customer use cases for this behavior. I'm open to restoring it if customers are actually using it, but would like to make sure the API is fully supporting whatever the need is.

Aug 5 2020, 6:55 PM
epriestley committed rP98e0440d459b: In 1-up source diffs, retain the "No newline at end of file" on "\" lines (authored by epriestley).
In 1-up source diffs, retain the "No newline at end of file" on "\" lines
Aug 5 2020, 5:17 PM
epriestley closed D21433: In 1-up source diffs, retain the "No newline at end of file" on "\" lines.
Aug 5 2020, 5:17 PM
epriestley triaged T13557: Unified diffs could do a better job of rendering omitted trailing newlines as Wishlist priority.
Aug 5 2020, 5:07 PM · Differential
epriestley requested review of D21433: In 1-up source diffs, retain the "No newline at end of file" on "\" lines.
Aug 5 2020, 4:50 PM
epriestley added a comment to D21225: Make the "attach_inlines" parameter to "differential.createcomment" a no-op.

I'm not aware of any current customer use cases for this behavior. I'm open to restoring it if customers are actually using it, but would like to make sure the API is fully supporting whatever the need is.

Aug 5 2020, 3:51 PM
artms added a comment to D21225: Make the "attach_inlines" parameter to "differential.createcomment" a no-op.

What is now programmatical way to "submit" unsubmitted inline comments? Some users are using it to post comments from bots, now if we use differential.createinline api method all bot comments are visible as unsubmitted and looks like there is no way to set it in stone...

Aug 5 2020, 11:39 AM

Aug 2 2020

Harbormaster failed to build B24817: rP287b7485a68a: (stable) Move "Wait for Previous Commits to Build" out of prototype for rP287b7485a68a: (stable) Move "Wait for Previous Commits to Build" out of prototype!
Aug 2 2020, 2:56 AM
epriestley committed rP287b7485a68a: (stable) Move "Wait for Previous Commits to Build" out of prototype (authored by epriestley).
(stable) Move "Wait for Previous Commits to Build" out of prototype
Aug 2 2020, 2:55 AM

Jul 30 2020

jg added a comment to D21432: Move "Wait for Previous Commits to Build" out of prototype.

lotsofthumbsup

Jul 30 2020, 7:56 PM
epriestley added a comment to D21432: Move "Wait for Previous Commits to Build" out of prototype.

See PHI1833.

Jul 30 2020, 7:50 PM
epriestley committed rPc1eeacd8500b: Move "Wait for Previous Commits to Build" out of prototype (authored by epriestley).
Move "Wait for Previous Commits to Build" out of prototype
Jul 30 2020, 7:46 PM
epriestley closed D21432: Move "Wait for Previous Commits to Build" out of prototype.
Jul 30 2020, 7:46 PM
epriestley requested review of D21432: Move "Wait for Previous Commits to Build" out of prototype.
Jul 30 2020, 7:45 PM

Jul 28 2020

epriestley requested review of D21431: Improve handle/status list display on devices in commit graph lists.
Jul 28 2020, 6:16 PM
epriestley added a revision to 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: D21431: Improve handle/status list display on devices in commit graph lists.
Jul 28 2020, 6:15 PM · Differential
epriestley requested review of D21430: Improve commit action item layout on mobile.
Jul 28 2020, 5:11 PM
epriestley added a revision to 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: D21430: Improve commit action item layout on mobile.
Jul 28 2020, 5:08 PM · Differential

Jul 27 2020

epriestley updated the task description for T13548: Upgrading: Mid 2020 Changes to "arc feature" / "arc branch" / "arc bookmark".
Jul 27 2020, 6:06 PM · Guides, Arcanist

Jul 25 2020

epriestley edited the content of Changelog.
Jul 25 2020, 3:26 PM
epriestley committed rARCccd39feb6d3b: (stable) Promote 2020 Week 29 (authored by epriestley).
(stable) Promote 2020 Week 29
Jul 25 2020, 3:25 PM
epriestley committed rP1b54f45c4c47: (stable) Promote 2020 Week 29 (authored by epriestley).
(stable) Promote 2020 Week 29
Jul 25 2020, 3:25 PM
epriestley created an object: 2020 Week 29 (Late July).
Jul 25 2020, 3:25 PM
avivey added a comment to T13550: Strip markup from project descriptions displayed in the "Browse Projects" dialog.

related to T3278, probably?

Jul 25 2020, 10:57 AM · Typeahead, Feature Request

Jul 24 2020

epriestley committed rP017ef1927c3e: Revert use of "user-select: all" to modify tab selection behavior (authored by epriestley).
Revert use of "user-select: all" to modify tab selection behavior
Jul 24 2020, 8:41 PM
epriestley added a reverting change for D21419: In source views, wrap display tabs in "user-select: all" to improve cursor selection behavior: rP017ef1927c3e: Revert use of "user-select: all" to modify tab selection behavior.
Jul 24 2020, 8:41 PM
epriestley closed D21429: Revert use of "user-select: all" to modify tab selection behavior.
Jul 24 2020, 8:41 PM
epriestley requested review of D21429: Revert use of "user-select: all" to modify tab selection behavior.
Jul 24 2020, 8:41 PM
epriestley added a comment to D21419: In source views, wrap display tabs in "user-select: all" to improve cursor selection behavior.

I reverted this in D21429, since it also makes these sequences select with a single click.

Jul 24 2020, 8:40 PM
epriestley added a reverting change for D21419: In source views, wrap display tabs in "user-select: all" to improve cursor selection behavior: D21429: Revert use of "user-select: all" to modify tab selection behavior.
Jul 24 2020, 8:40 PM
epriestley added a comment to T13556: Setting "phd.trace" may cause TTY testing with "posix_isatty()" against closed pipes.

In D21426, I removed phd.trace and phd.verbose.

Jul 24 2020, 12:18 AM · Daemons

Jul 23 2020

epriestley committed rPa27c83757d9f: Remove ancient "phd.trace" and "phd.verbose" configuration options (authored by epriestley).
Remove ancient "phd.trace" and "phd.verbose" configuration options
Jul 23 2020, 7:31 PM
epriestley closed D21426: Remove ancient "phd.trace" and "phd.verbose" configuration options.
Jul 23 2020, 7:31 PM
epriestley updated the diff for D21426: Remove ancient "phd.trace" and "phd.verbose" configuration options.
  • Include "obsolete config" setup checks.
Jul 23 2020, 7:28 PM
epriestley requested review of D21426: Remove ancient "phd.trace" and "phd.verbose" configuration options.
Jul 23 2020, 7:23 PM
epriestley added a revision to T13556: Setting "phd.trace" may cause TTY testing with "posix_isatty()" against closed pipes: D21426: Remove ancient "phd.trace" and "phd.verbose" configuration options.
Jul 23 2020, 7:22 PM · Daemons
epriestley committed rP78d1b62bb8e9: Streamline handling of Futures and PIDs in daemons (authored by epriestley).
Streamline handling of Futures and PIDs in daemons
Jul 23 2020, 6:22 PM
epriestley committed rP5f0535934d72: Manage PIDs more carefully in DaemonHandle (authored by epriestley).
Manage PIDs more carefully in DaemonHandle
Jul 23 2020, 6:22 PM
epriestley closed D21424: Manage PIDs more carefully in DaemonHandle.
Jul 23 2020, 6:22 PM
epriestley closed D21425: Streamline handling of Futures and PIDs in daemons.
Jul 23 2020, 6:22 PM
epriestley committed rARCceb082ef6b29: Give Futures clearer start/end and exception semantics (authored by epriestley).
Give Futures clearer start/end and exception semantics
Jul 23 2020, 6:22 PM
epriestley closed D21423: Give Futures clearer start/end and exception semantics.
Jul 23 2020, 6:22 PM
epriestley claimed T13556: Setting "phd.trace" may cause TTY testing with "posix_isatty()" against closed pipes.
Jul 23 2020, 6:05 PM · Daemons
epriestley triaged T13556: Setting "phd.trace" may cause TTY testing with "posix_isatty()" against closed pipes as Wishlist priority.
Jul 23 2020, 6:05 PM · Daemons
epriestley updated the diff for D21425: Streamline handling of Futures and PIDs in daemons.
  • Use "$caught", not "$ex", to make the code less fragile.
Jul 23 2020, 5:42 PM