Page MenuHomePhabricator
Feed All Stories

Aug 19 2021

0 created T13663: TypeError when editing calendar import.
Aug 19 2021, 2:27 AM · Calendar

Aug 18 2021

epriestley removed a member for Phacility High Command: amckinley.
Aug 18 2021, 7:31 PM

Aug 17 2021

cspeckmim awarded T13133: Phabricator Query Layer Overview a Mountain of Wealth token.
Aug 17 2021, 12:14 AM · Guides

Aug 2 2021

epriestley committed rP8daaf5ef2145: Pass a real context object to Phriction previews, fixing mentions (authored by epriestley).
Pass a real context object to Phriction previews, fixing mentions
Aug 2 2021, 3:36 PM
epriestley closed T13662: Mentions break Phriction preview as Resolved by committing rP8daaf5ef2145: Pass a real context object to Phriction previews, fixing mentions.
Aug 2 2021, 3:36 PM · Phriction
epriestley closed D21709: Pass a real context object to Phriction previews, fixing mentions.
Aug 2 2021, 3:36 PM
epriestley closed D21708: Fix Phriction document previews for the root document ("/") with Apache option "MergeSlashes On".
Aug 2 2021, 3:36 PM
epriestley committed rPc7550dbee9b1: Fix Phriction document previews for the root document ("/") with Apache option… (authored by epriestley).
Fix Phriction document previews for the root document ("/") with Apache option…
Aug 2 2021, 3:36 PM
epriestley added a comment to T13652: Notes on Ardunio CNC drivers.

I found this channel incredibly helpful for repairing/replacing components on boards:

Aug 2 2021, 4:19 AM

Aug 1 2021

0 accepted D21709: Pass a real context object to Phriction previews, fixing mentions.
Aug 1 2021, 7:11 PM
epriestley added a comment to D21709: Pass a real context object to Phriction previews, fixing mentions.

You can render remarkup using this rule without having a meaningful relative base URI, e.g. on some other object type or via remarkup.process in the API. This isn't necessarily a meaningful operation and the result may not be useful, but remarkup generally tries not to fail loudly: if you copy/paste a block of text from somewhere that happens to have some substrings which aren't valid, the desired behavior is generally for your text to be processed in some best-guess-at-sensible way -- and usually emitted unmodified (see also discussion in D21713).

Aug 1 2021, 7:06 PM
0 added inline comments to D21709: Pass a real context object to Phriction previews, fixing mentions.
Aug 1 2021, 6:51 PM
0 accepted D21708: Fix Phriction document previews for the root document ("/") with Apache option "MergeSlashes On".

This also fixes root document preview in lighttpd, which seems to have mandatory slash merging.

Aug 1 2021, 6:08 PM

Jul 29 2021

epriestley committed rPdbe2fb466fed: Add an "eval" rule to Remarkup (authored by epriestley).
Add an "eval" rule to Remarkup
Jul 29 2021, 11:47 PM
epriestley closed D21713: Add an "eval" rule to Remarkup.
Jul 29 2021, 11:47 PM
epriestley added inline comments to D21713: Add an "eval" rule to Remarkup.
Jul 29 2021, 8:31 PM
cspeckmim accepted D21713: Add an "eval" rule to Remarkup.
Jul 29 2021, 8:30 PM
epriestley updated the diff for D21713: Add an "eval" rule to Remarkup.
  • For now, just return the literal input if we fail to evaluate an expression.
Jul 29 2021, 8:18 PM
epriestley added a comment to D21713: Add an "eval" rule to Remarkup.

When intent is ambiguous (the user might or might not be trying to invoke a Remarkup rule), I try to make the output of an "invalid" input exactly the same as the input, so (for example) copy/pasting text into Phabricator doesn't mangle it into a big blob of nonsense just because you happened to have some magic words in there.

Jul 29 2021, 3:57 PM
cspeckmim added a comment to D21713: Add an "eval" rule to Remarkup.

I think _() is the pht() of Gnu "gettext". Modern programers may mostly be more familiar with Python than with gettext, of course.

Oh yea that's right. A long time ago I worked on a python web application that used this was probably confusing it.

Jul 29 2021, 1:03 AM
cspeckmim planned changes to D21712: Trying out removing "Phabricator" from some user-visible text.

I'll plan to get this on a test instance and get some screenshots

Jul 29 2021, 1:01 AM
cspeckmim added inline comments to D21713: Add an "eval" rule to Remarkup.
Jul 29 2021, 12:57 AM
cspeckmim updated the diff for D21712: Trying out removing "Phabricator" from some user-visible text.

Try escaping the ${{{

Jul 29 2021, 12:54 AM
epriestley added a comment to D21713: Add an "eval" rule to Remarkup.

I suspect escaping things in PHP will be pretty rare and that the "collides with PHP strings" downside will be very small.

Jul 29 2021, 12:24 AM

Jul 28 2021

cspeckmim added a comment to D21713: Add an "eval" rule to Remarkup.

You can either escape ${{{strings.x.y}}} as \${{{strings.x.y}}} or suggest a different syntax for the "eval" rule --- I'm not married to ${{{...}}}.

I think this format makes sense to me. Nothing else really comes to mind. The squiggles are keeping consistency with other remarkup rules like figlet and cowsay. I think I've seen $ used in other places (and languages) with some relation to string substitution. I think python convention uses _, so another possibility could be _{{{...}}}.

Jul 28 2021, 10:18 PM
epriestley added a comment to D21712: Trying out removing "Phabricator" from some user-visible text.

Ah, yeah, the build issue is that ${...} in a PHP double-quoted string is semantic, so PHP is trying to do something with ${{{...}}} and failing with a syntax exception during parsing. You can either escape ${{{strings.x.y}}} as \${{{strings.x.y}}} or suggest a different syntax for the "eval" rule --- I'm not married to ${{{...}}}.

Jul 28 2021, 5:13 AM
epriestley added a comment to D21712: Trying out removing "Phabricator" from some user-visible text.

That build failure may be related to recent changes to T13072, or something else server-side -- I don't expect build failures to look like that in Harbormaster. Let me see if I can reproduce it locally.

Jul 28 2021, 5:09 AM
cspeckmim added a comment to D21680: An assortment of fixes and updates to using arc-land with mercurial.

I’ve used this a few times in the past week and the behavior is overall better for my own case. I had one other developer upgrade to master branch and also reported better behavior, though his setup is very similar to my own.

Jul 28 2021, 4:22 AM
cspeckmim added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

Adding some feedback regarding these changes (I should’ve made a task for this first). Last week I had two developers separately approach me with issues they ran into with arc diff. I had them upgrade to the master branch and this resolved their issues.

Jul 28 2021, 4:20 AM
cspeckmim added inline comments to D21712: Trying out removing "Phabricator" from some user-visible text.
Jul 28 2021, 4:07 AM
cspeckmim planned changes to D21712: Trying out removing "Phabricator" from some user-visible text.

I'll investigate the build failures -- I suspect it's the use of the new remarkup rule. At the moment I'm working from a Windows system and haven't been able to get xhpast or unit tests running properly.

Jul 28 2021, 4:01 AM
Harbormaster failed remote builds in B25527: Diff 51727 for D21712: Trying out removing "Phabricator" from some user-visible text!
Jul 28 2021, 3:59 AM
cspeckmim updated the diff for D21712: Trying out removing "Phabricator" from some user-visible text.

Minor corrections

Jul 28 2021, 3:59 AM
Harbormaster failed remote builds in B25526: Diff 51726 for D21712: Trying out removing "Phabricator" from some user-visible text!
Jul 28 2021, 3:55 AM
cspeckmim updated the diff for D21712: Trying out removing "Phabricator" from some user-visible text.

Preferring "installation" as a generic term, where possible

Jul 28 2021, 3:54 AM
cspeckmim accepted D21713: Add an "eval" rule to Remarkup.
Jul 28 2021, 3:11 AM
cspeckmim added a comment to D21713: Add an "eval" rule to Remarkup.

This looks pretty cool and will be useful for maintaining existing text that goes through remarkup. Really useful doc on PhutilRemarkupBlockStorage btw.

Jul 28 2021, 3:10 AM

Jul 27 2021

epriestley requested review of D21713: Add an "eval" rule to Remarkup.
Jul 27 2021, 9:08 PM
epriestley added a revision to T13658: How to rebrand Phabricator: D21713: Add an "eval" rule to Remarkup.
Jul 27 2021, 9:07 PM
epriestley committed rPcc8cbed02941: Make "DifferentialDiff->properties" a proper "attachable" property (authored by epriestley).
Make "DifferentialDiff->properties" a proper "attachable" property
Jul 27 2021, 8:39 PM
epriestley closed D21710: Make "DifferentialDiff->properties" a proper "attachable" property.
Jul 27 2021, 8:39 PM
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