Page MenuHomePhabricator
Feed All Stories

Jul 27 2021

epriestley requested changes to D21712: Trying out removing "Phabricator" from some user-visible text.

These generally look reasonable to me, some thoughts inline. In some of these cases I think we can probably just remove extra references to "this install", "this install of Phabricator on this server", etc., and let it be implied by context.

Jul 27 2021, 8:36 PM
cspeckmim published D21712: Trying out removing "Phabricator" from some user-visible text for review.

This is just trying out some changes that modify the language to reduce instances of "Phabricator" from user-visible text. I went through the auth provider implementations to try out using "platform", "server", "service", and in one case I think just "instance".

Jul 27 2021, 12:51 AM
cspeckmim added a revision to T13658: How to rebrand Phabricator: D21712: Trying out removing "Phabricator" from some user-visible text.
Jul 27 2021, 12:44 AM

Jul 26 2021

epriestley committed rARC82016c00e132: Name extension as "arc-hg", not "arg-hg" (authored by epriestley).
Name extension as "arc-hg", not "arg-hg"
Jul 26 2021, 7:27 PM
epriestley closed D21711: Name extension as "arc-hg", not "arg-hg".
Jul 26 2021, 7:27 PM
epriestley updated the summary of D21708: Fix Phriction document previews for the root document ("/") with Apache option "MergeSlashes On".
Jul 26 2021, 7:27 PM
cspeckmim accepted D21711: Name extension as "arc-hg", not "arg-hg".

argh

Jul 26 2021, 6:52 PM
epriestley closed T13659: `arc land` may fail with missing rebase extension as Resolved.

I caught one cosmetic issue in D21711, but this works properly for me locally now. Thanks!

Jul 26 2021, 6:49 PM · Arcanist, Mercurial
epriestley requested review of D21711: Name extension as "arc-hg", not "arg-hg".
Jul 26 2021, 6:49 PM
epriestley added a revision to T13659: `arc land` may fail with missing rebase extension: D21711: Name extension as "arc-hg", not "arg-hg".
Jul 26 2021, 6:49 PM · Arcanist, Mercurial
epriestley requested review of D21710: Make "DifferentialDiff->properties" a proper "attachable" property.
Jul 26 2021, 6:45 PM
epriestley requested review of D21709: Pass a real context object to Phriction previews, fixing mentions.
Jul 26 2021, 6:33 PM
epriestley added a revision to T13662: Mentions break Phriction preview: D21709: Pass a real context object to Phriction previews, fixing mentions.
Jul 26 2021, 6:32 PM · Phriction
epriestley requested review of D21708: Fix Phriction document previews for the root document ("/") with Apache option "MergeSlashes On".
Jul 26 2021, 6:18 PM
epriestley added a revision to T13662: Mentions break Phriction preview: D21708: Fix Phriction document previews for the root document ("/") with Apache option "MergeSlashes On".
Jul 26 2021, 6:17 PM · Phriction
epriestley added a comment to T13662: Mentions break Phriction preview.

I ran into this while trying to reproduce the issue:

Jul 26 2021, 6:13 PM · Phriction
epriestley claimed T13662: Mentions break Phriction preview.
Jul 26 2021, 5:56 PM · Phriction

Jul 22 2021

0 updated subscribers of T13662: Mentions break Phriction preview.
Jul 22 2021, 4:23 AM · Phriction
0 created T13662: Mentions break Phriction preview.
Jul 22 2021, 4:20 AM · Phriction

Jul 21 2021

epriestley added a comment to T13648: Transaction publishing may stall during mail expansion of package recipients.

The stalled transactions on this host published after I deployed the update.

Jul 21 2021, 11:39 PM · Transactions
epriestley removed a reviewer for D21498: Fix error handler on PHP 8: Unknown Object (Owners Package).
Jul 21 2021, 11:36 PM
epriestley removed a reviewer for D21497: Suppress PHP 8 deprecation warning in __arcanist_init_script__: Unknown Object (Owners Package).
Jul 21 2021, 11:34 PM
epriestley committed rP387d3b4983d7: Resolve deleted packages properly as having no mailable members (authored by epriestley).
Resolve deleted packages properly as having no mailable members
Jul 21 2021, 9:50 PM
epriestley closed D21707: Resolve deleted packages properly as having no mailable members.
Jul 21 2021, 9:50 PM
epriestley closed T13648: Transaction publishing may stall during mail expansion of package recipients as Resolved by committing rP387d3b4983d7: Resolve deleted packages properly as having no mailable members.
Jul 21 2021, 9:50 PM · Transactions
epriestley requested review of D21707: Resolve deleted packages properly as having no mailable members.
Jul 21 2021, 9:27 PM
epriestley updated the task description for T13661: Give Phame configurable interact policies.
Jul 21 2021, 9:26 PM · Phacility, Phame
epriestley triaged T13661: Give Phame configurable interact policies as Low priority.
Jul 21 2021, 9:25 PM · Phacility, Phame
epriestley closed D21687: Correct the flow of edit authority when sending messages to HarbormasterBuild objects.
Jul 21 2021, 9:24 PM
epriestley added a revision to T13648: Transaction publishing may stall during mail expansion of package recipients: D21707: Resolve deleted packages properly as having no mailable members.
Jul 21 2021, 9:22 PM · Transactions
epriestley closed D21696: Add a side nav to Conduit API method console pages.
Jul 21 2021, 9:19 PM
epriestley closed D21699: Rename "HarbormasterRestartException" to "HarbormasterMessageException".
Jul 21 2021, 9:18 PM
epriestley closed D21695: Add stub "harbormaster.build.edit" and "harbormaster.buildable.edit" API methods.
Jul 21 2021, 9:18 PM
epriestley committed rP3df1e175273f: Add a side nav to Conduit API method console pages (authored by epriestley).
Add a side nav to Conduit API method console pages
Jul 21 2021, 9:18 PM
epriestley closed D21698: Allow "harbormaster.sendmessage" to send control command (pause, restart, abort, resume) to Builds/Buildables.
Jul 21 2021, 9:18 PM
epriestley committed rP2ff1d4b3b007: Add stub "harbormaster.build.edit" and "harbormaster.buildable.edit" API methods (authored by epriestley).
Add stub "harbormaster.build.edit" and "harbormaster.buildable.edit" API methods
Jul 21 2021, 9:18 PM
epriestley committed rP4cffaa600bec: Rename "HarbormasterRestartException" to "HarbormasterMessageException" (authored by epriestley).
Rename "HarbormasterRestartException" to "HarbormasterMessageException"
Jul 21 2021, 9:18 PM
epriestley committed rP4c4123f98b6d: Allow "harbormaster.sendmessage" to send control command (pause, restart, abort… (authored by epriestley).
Allow "harbormaster.sendmessage" to send control command (pause, restart, abort…
Jul 21 2021, 9:18 PM
epriestley committed rP346ffc51e100: Modularize "HarbormasterBuildableTransaction" (authored by epriestley).
Modularize "HarbormasterBuildableTransaction"
Jul 21 2021, 9:18 PM
epriestley closed D21694: Modularize "HarbormasterBuildableTransaction".
Jul 21 2021, 9:18 PM
epriestley closed D21693: Remove "HarbormasterBuildableTransaction::TYPE_CREATE".
Jul 21 2021, 9:18 PM
epriestley committed rP706b91adf677: Remove "HarbormasterBuildableTransaction::TYPE_CREATE" (authored by epriestley).
Remove "HarbormasterBuildableTransaction::TYPE_CREATE"
Jul 21 2021, 9:18 PM
epriestley committed rP1a1395579f95: Remove "HarbormasterBuildCommand" (authored by epriestley).
Remove "HarbormasterBuildCommand"
Jul 21 2021, 9:18 PM
epriestley closed D21692: Remove "HarbormasterBuildCommand".
Jul 21 2021, 9:18 PM
epriestley closed D21691: Modularize almost all Harbormaster build message workflows and UI/UX.
Jul 21 2021, 9:18 PM
epriestley committed rP1212dc5fbe8b: Modularize almost all Harbormaster build message workflows and UI/UX (authored by epriestley).
Modularize almost all Harbormaster build message workflows and UI/UX
Jul 21 2021, 9:18 PM
epriestley committed rP8bbee921398d: Modularize individual Harbormaster build messages (authored by epriestley).
Modularize individual Harbormaster build messages
Jul 21 2021, 9:17 PM
epriestley closed D21690: Modularize individual Harbormaster build messages.
Jul 21 2021, 9:17 PM
epriestley closed D21689: Modularize HarbormasterBuildTransaction.
Jul 21 2021, 9:17 PM
epriestley committed rP6dfea0adaddc: Modularize HarbormasterBuildTransaction (authored by epriestley).
Modularize HarbormasterBuildTransaction
Jul 21 2021, 9:17 PM
epriestley committed rPb84c02959a8b: Remove "HarbormasterBuildTransaction::TYPE_CREATE" (authored by epriestley).
Remove "HarbormasterBuildTransaction::TYPE_CREATE"
Jul 21 2021, 9:17 PM
epriestley committed rP012af00731fd: Improve formality of "HarbormasterBuild" states (authored by epriestley).
Improve formality of "HarbormasterBuild" states
Jul 21 2021, 9:17 PM
epriestley committed rPbf83fffca129: Correct the flow of edit authority when sending messages to HarbormasterBuild… (authored by epriestley).
Correct the flow of edit authority when sending messages to HarbormasterBuild…
Jul 21 2021, 9:17 PM
epriestley closed D21688: Remove "HarbormasterBuildTransaction::TYPE_CREATE".
Jul 21 2021, 9:17 PM
epriestley closed D21685: Improve formality of "HarbormasterBuild" states.
Jul 21 2021, 9:17 PM
epriestley committed rPb48d4fabaf9e: Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage" (authored by epriestley).
Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage"
Jul 21 2021, 9:17 PM
epriestley closed D21684: Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage".
Jul 21 2021, 9:17 PM
epriestley committed rP0f93d1ffe450: Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages"… (authored by epriestley).
Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages"…
Jul 21 2021, 9:17 PM
epriestley closed D21683: Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages" table.
Jul 21 2021, 9:17 PM
cspeckmim closed D21706: Update other usages of "hg rebase" to use the new extension-enabling function.
Jul 21 2021, 9:11 PM
cspeckmim committed rARC76a2976fd9a4: Update other usages of "hg rebase" to use the new extension-enabling function (authored by cspeckmim).
Update other usages of "hg rebase" to use the new extension-enabling function
Jul 21 2021, 9:11 PM
cspeckmim updated the test plan for D21706: Update other usages of "hg rebase" to use the new extension-enabling function.
Jul 21 2021, 9:11 PM
epriestley accepted D21706: Update other usages of "hg rebase" to use the new extension-enabling function.
Jul 21 2021, 9:06 PM
cspeckmim updated the diff for D21706: Update other usages of "hg rebase" to use the new extension-enabling function.

These keep springing up

Jul 21 2021, 9:03 PM
cspeckmim requested review of D21706: Update other usages of "hg rebase" to use the new extension-enabling function.
Jul 21 2021, 8:59 PM
cspeckmim added a revision to T13659: `arc land` may fail with missing rebase extension: D21706: Update other usages of "hg rebase" to use the new extension-enabling function.
Jul 21 2021, 8:59 PM · Arcanist, Mercurial
Harbormaster failed to build B25495: rPaf6cc0c934c4: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in… for rPaf6cc0c934c4: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in…!
Jul 21 2021, 5:22 PM
epriestley committed rARCec68005d758c: Remove "phutil_deprecated()" (authored by epriestley).
Remove "phutil_deprecated()"
Jul 21 2021, 5:21 PM
epriestley closed D21704: Remove "phutil_deprecated()".
Jul 21 2021, 5:21 PM
epriestley closed D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:21 PM
epriestley committed rARC8bb7d5889050: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()" (authored by epriestley).
Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()"
Jul 21 2021, 5:21 PM
epriestley closed T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()" as Resolved by committing rARC8bb7d5889050: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:21 PM · Infrastructure
epriestley committed rPaf6cc0c934c4: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in… (authored by epriestley).
Use "resolve()", not "execute()", for PhutilExecPassthru callsites in…
Jul 21 2021, 5:21 PM
epriestley closed D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 5:21 PM
epriestley added inline comments to D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:20 PM
cspeckmim accepted D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 5:18 PM
cspeckmim accepted D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:17 PM
epriestley updated the test plan for D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:12 PM
epriestley requested review of D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:12 PM
epriestley added a revision to T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()": D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:11 PM · Infrastructure
epriestley requested review of D21704: Remove "phutil_deprecated()".
Jul 21 2021, 5:08 PM
epriestley requested review of D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 4:56 PM
epriestley added a revision to T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()": D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 4:54 PM · Infrastructure
epriestley triaged T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()" as Low priority.
Jul 21 2021, 4:49 PM · Infrastructure
epriestley added a comment to T13659: `arc land` may fail with missing rebase extension.

(After picking up D21697, the approach works properly in my environment where I originally ran into this, but there are a couple of rebase calls still present in ArcanistMercurialLandEngine that don't use the new API yet.)

Jul 21 2021, 4:44 PM · Arcanist, Mercurial
epriestley added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

I just use TextMate, which is a fairly lightweight editor. I have it minimally configured to disable a couple of default behaviors and add a couple aliases, and I have a crude "jump-to-definition" script set up that tries to open whatever file defines the class/function under the cursor and succeeds about 95% of the time.

Jul 21 2021, 4:29 PM
cspeckmim added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

Do you use an editor like PHPStorm or WebStorm? So far I've been using SublimeText and ripgrep as the primarily tools.

Jul 21 2021, 4:56 AM
cspeckmim closed D21697: Refactor how Mercurial runs commands that require extensions.
Jul 21 2021, 4:40 AM
cspeckmim committed rARCac365c3ee509: Refactor how Mercurial runs commands that require extensions (authored by cspeckmim).
Refactor how Mercurial runs commands that require extensions
Jul 21 2021, 4:40 AM
cspeckmim updated the test plan for D21697: Refactor how Mercurial runs commands that require extensions.
Jul 21 2021, 4:19 AM
epriestley added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

...the one with --onto-remote default not appearing in the trace output...

Jul 21 2021, 4:14 AM
cspeckmim added inline comments to D21697: Refactor how Mercurial runs commands that require extensions.
Jul 21 2021, 4:10 AM
cspeckmim added inline comments to D21697: Refactor how Mercurial runs commands that require extensions.
Jul 21 2021, 4:05 AM
epriestley added inline comments to D21697: Refactor how Mercurial runs commands that require extensions.
Jul 21 2021, 4:04 AM
cspeckmim updated the diff for D21697: Refactor how Mercurial runs commands that require extensions.
  • Updated the MercurialMarkerQuery to use a similar execPassthruWithExtension(). This results in printing the execution with --trace and better encapsulates the functions handling the extension-enabling details.
  • Made newConfiguredFuture() to combine the environment/cwd configuration for mercurial executions.
  • Updated to store result of func_get_args() into a variable first instead of passing directly into a function call.
  • Updated comments.
Jul 21 2021, 4:01 AM
cspeckmim added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

Oh I forgot to include that it does launch the regular local version of arc-ls-markers in my sample output, which may have provided more context for my comment about the one with --onto-remote default not appearing in the trace output

Jul 21 2021, 3:37 AM
cspeckmim added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

...or did you mean something else?

Specifically the arc-ls-markers which uses the --output argument, which appears to only be used when a remote marker is specified for landing. That uses $api->newPassthru(). When checking the output from arc land --onto-remote default it doesn't appear to log the execution of the passthru.

Jul 21 2021, 3:36 AM
epriestley committed rARC35c1b9bf027b: Fix an ExecFuture typo, "preprebuilt" (authored by epriestley).
Fix an ExecFuture typo, "preprebuilt"
Jul 21 2021, 3:18 AM
epriestley closed D21702: Fix an ExecFuture typo, "preprebuilt".
Jul 21 2021, 3:18 AM
epriestley requested review of D21702: Fix an ExecFuture typo, "preprebuilt".
Jul 21 2021, 3:15 AM