Page MenuHomePhabricator
Feed All Stories

May 23 2019

epriestley moved T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained from Backlog to Next on the Remarkup board.
May 23 2019, 2:46 PM · Documentation, Remarkup
epriestley moved T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion from Backlog to Next on the Remarkup board.
May 23 2019, 2:45 PM · Remarkup, Diffusion
epriestley edited projects for T9825: Remarkup is rendered literally in Diviner, added: Diviner; removed Remarkup.
May 23 2019, 2:45 PM · Diviner
epriestley removed a project from T12973: Commit list changes based on hash in URL: Remarkup.
May 23 2019, 2:44 PM · Diffusion
asherkin added a comment to D19681: Fix arc land on odd/modern git-svn checkouts.

Thanks for the review - I dropped in a comment referencing the task when committing so it's clear why this is here when the refactoring comes along 🙂

May 23 2019, 10:01 AM
asherkin committed rARC7329bc7c32b9: Fix arc land on odd/modern git-svn checkouts (authored by asherkin).
Fix arc land on odd/modern git-svn checkouts
May 23 2019, 9:58 AM
asherkin closed D19681: Fix arc land on odd/modern git-svn checkouts.
May 23 2019, 9:58 AM
asherkin closed T13293: `arc land` using git-svn fails in certain configurations as Resolved by committing rARC7329bc7c32b9: Fix arc land on odd/modern git-svn checkouts.
May 23 2019, 9:58 AM · Subversion, Git, Arcanist

May 22 2019

epriestley committed rP2e2dc47f0763: In the Herald test console, don't consider transactions that Herald rules… (authored by epriestley).
In the Herald test console, don't consider transactions that Herald rules…
May 22 2019, 11:34 PM
epriestley closed D20547: In the Herald test console, don't consider transactions that Herald rules applied.
May 22 2019, 11:34 PM
epriestley committed rP31e623afcc92: Use the same transaction group ID for transactions applied indirectly by a sub… (authored by epriestley).
Use the same transaction group ID for transactions applied indirectly by a sub…
May 22 2019, 11:33 PM
epriestley closed D20546: Use the same transaction group ID for transactions applied indirectly by a sub-editor.
May 22 2019, 11:33 PM
epriestley committed rPf6af1c437424: When creating a Phriction document, mark initial transactions as "create"… (authored by epriestley).
When creating a Phriction document, mark initial transactions as "create"…
May 22 2019, 11:28 PM
epriestley closed D20548: When creating a Phriction document, mark initial transactions as "create" transactions to fix weird email.
May 22 2019, 11:28 PM
epriestley committed rPb95bf722d55e: Drop the "update revision with commit diff" transaction if the revision is… (authored by epriestley).
Drop the "update revision with commit diff" transaction if the revision is…
May 22 2019, 11:26 PM
epriestley closed D20545: Drop the "update revision with commit diff" transaction if the revision is already closed.
May 22 2019, 11:26 PM
epriestley committed rP1eff4fdca332: Prevent "Differential Revision: ..." from counting as a mention in commit… (authored by epriestley).
Prevent "Differential Revision: ..." from counting as a mention in commit…
May 22 2019, 11:22 PM
epriestley closed D20544: Prevent "Differential Revision: ..." from counting as a mention in commit messages.
May 22 2019, 11:22 PM
epriestley committed rPfa4dcaa3aaff: Stabilize sorting of feed stories with similar strength (authored by epriestley).
Stabilize sorting of feed stories with similar strength
May 22 2019, 10:51 PM
epriestley closed D20540: Stabilize sorting of feed stories with similar strength.
May 22 2019, 10:51 PM
amckinley accepted D20548: When creating a Phriction document, mark initial transactions as "create" transactions to fix weird email.
May 22 2019, 10:39 PM
amckinley accepted D20545: Drop the "update revision with commit diff" transaction if the revision is already closed.
May 22 2019, 10:36 PM
amckinley accepted D20546: Use the same transaction group ID for transactions applied indirectly by a sub-editor.
May 22 2019, 10:30 PM
amckinley accepted D20547: In the Herald test console, don't consider transactions that Herald rules applied.
May 22 2019, 10:30 PM
epriestley requested review of D20548: When creating a Phriction document, mark initial transactions as "create" transactions to fix weird email.
May 22 2019, 10:17 PM
epriestley added a revision to T13289: Plans: 2019 Week 21-23 Bonus Content: D20548: When creating a Phriction document, mark initial transactions as "create" transactions to fix weird email.
May 22 2019, 10:15 PM · Plans
epriestley added a comment to D20546: Use the same transaction group ID for transactions applied indirectly by a sub-editor.

D20547 alone would also fix this specific problem (by skipping the Herald transactions), but I think this change is good on its own and generally makes behavior more consistent/expected.

May 22 2019, 9:50 PM
epriestley requested review of D20547: In the Herald test console, don't consider transactions that Herald rules applied.
May 22 2019, 9:49 PM
epriestley added a revision to T13283: When Herald acts, include the transaction group as part of the "state" it acts upon: D20547: In the Herald test console, don't consider transactions that Herald rules applied.
May 22 2019, 9:48 PM · Herald
epriestley requested review of D20546: Use the same transaction group ID for transactions applied indirectly by a sub-editor.
May 22 2019, 9:43 PM
epriestley added a revision to T13283: When Herald acts, include the transaction group as part of the "state" it acts upon: D20546: Use the same transaction group ID for transactions applied indirectly by a sub-editor.
May 22 2019, 9:41 PM · Herald
amckinley accepted D20544: Prevent "Differential Revision: ..." from counting as a mention in commit messages.
May 22 2019, 9:37 PM
epriestley requested review of D20545: Drop the "update revision with commit diff" transaction if the revision is already closed.
May 22 2019, 9:32 PM
epriestley added a revision to T13290: Clean up remaining "Autoclose" behaviors related to "One Revision, Many Commits": D20545: Drop the "update revision with commit diff" transaction if the revision is already closed.
May 22 2019, 9:31 PM · Diffusion, Differential
amckinley added a comment to D20540: Stabilize sorting of feed stories with similar strength.

Reading PHI1222; nice catch!

May 22 2019, 9:19 PM
amckinley accepted D20540: Stabilize sorting of feed stories with similar strength.
May 22 2019, 9:18 PM
epriestley requested review of D20544: Prevent "Differential Revision: ..." from counting as a mention in commit messages.
May 22 2019, 9:16 PM
amckinley updated the summary of D20540: Stabilize sorting of feed stories with similar strength.
May 22 2019, 9:15 PM
epriestley added a revision to T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion: D20544: Prevent "Differential Revision: ..." from counting as a mention in commit messages.
May 22 2019, 9:15 PM · Remarkup, Diffusion
epriestley added a revision to T13290: Clean up remaining "Autoclose" behaviors related to "One Revision, Many Commits": D20544: Prevent "Differential Revision: ..." from counting as a mention in commit messages.
May 22 2019, 9:15 PM · Diffusion, Differential
epriestley added a comment to T13283: When Herald acts, include the transaction group as part of the "state" it acts upon.

Currently, I think the primary transactions and transactions triggered by Herald are getting different group IDs. They should be the same group ID, e.g. all these transactions should be in the same "group" for the purposes of collecting transactions into effect groups:

May 22 2019, 7:56 PM · Herald
epriestley added a comment to T13290: Clean up remaining "Autoclose" behaviors related to "One Revision, Many Commits".

(This is mostly for my own notes and contains roughly zero insight.)

May 22 2019, 6:07 PM · Diffusion, Differential
epriestley added a project to T13292: Create a "Sounds" application: Sounds.
May 22 2019, 4:56 PM · Sounds
epriestley created Sounds.
May 22 2019, 4:56 PM
epriestley added a comment to T13292: Create a "Sounds" application.

If we were just doing personal notification sounds, I think there'd be a good argument for making this a Settings panel.

May 22 2019, 4:54 PM · Sounds
epriestley triaged T13293: `arc land` using git-svn fails in certain configurations as Low priority.
May 22 2019, 4:30 PM · Subversion, Git, Arcanist
epriestley added projects to T13293: `arc land` using git-svn fails in certain configurations: Arcanist, Git, Subversion.
May 22 2019, 4:30 PM · Subversion, Git, Arcanist
epriestley accepted D19681: Fix arc land on odd/modern git-svn checkouts.

Thanks, T13293 makes this much clearer.

May 22 2019, 4:30 PM
epriestley added a comment to T13294: Provide a transaction-level view of feed.

Per D20533, the major query this UI uses is currently unkeyed (no dateCreated key on transaction tables).

May 22 2019, 3:44 PM · Feed, Transactions
epriestley committed rPf91bef64f163: Stack chart functions in a more physical way (authored by epriestley).
Stack chart functions in a more physical way
May 22 2019, 12:40 PM
epriestley closed D20506: Stack chart functions in a more physical way.
May 22 2019, 12:40 PM
epriestley committed rPf190c42bcd2e: Store charts earlier and build them out a little later (authored by epriestley).
Store charts earlier and build them out a little later
May 22 2019, 12:39 PM
epriestley closed D20504: Store charts earlier and build them out a little later.
May 22 2019, 12:39 PM
epriestley committed rP493a6b72c1c0: Automatically select the range for charts in a general way (authored by epriestley).
Automatically select the range for charts in a general way
May 22 2019, 12:37 PM
epriestley closed D20503: Automatically select the range for charts in a general way.
May 22 2019, 12:37 PM
epriestley committed rPe90360c28967: Wrap "<min, max>" chart domain pairs in an "Interval" class (authored by epriestley).
Wrap "<min, max>" chart domain pairs in an "Interval" class
May 22 2019, 12:29 PM
epriestley closed D20502: Wrap "<min, max>" chart domain pairs in an "Interval" class.
May 22 2019, 12:29 PM
epriestley committed rPa80426b339c4: Provide chart function labels over the wire instead of making them up (authored by epriestley).
Provide chart function labels over the wire instead of making them up
May 22 2019, 12:23 PM
epriestley closed D20501: Provide chart function labels over the wire instead of making them up.
May 22 2019, 12:23 PM
epriestley added inline comments to D20501: Provide chart function labels over the wire instead of making them up.
May 22 2019, 12:22 PM
epriestley committed rPc6052b41a604: Label important data on charts (authored by epriestley).
Label important data on charts
May 22 2019, 12:21 PM
epriestley closed D20500: Label important data on charts.
May 22 2019, 12:21 PM
epriestley committed rP81456db5594e: Roughly support stacked area charts (authored by epriestley).
Roughly support stacked area charts
May 22 2019, 12:19 PM
epriestley closed D20498: Roughly support stacked area charts.
May 22 2019, 12:19 PM
epriestley committed rP0776b5ca2c4d: Update D3 to the current version (authored by epriestley).
Update D3 to the current version
May 22 2019, 12:16 PM
epriestley closed D20497: Update D3 to the current version.
May 22 2019, 12:16 PM
epriestley committed rP5c1b91ab457d: Consolidate burndown logic into a "BurndownChartEngine" (authored by epriestley).
Consolidate burndown logic into a "BurndownChartEngine"
May 22 2019, 12:10 PM
epriestley closed D20496: Consolidate burndown logic into a "BurndownChartEngine".
May 22 2019, 12:10 PM
epriestley committed rP0aee3da19e6a: Add a "Reports" menu item to Projects (authored by epriestley).
Add a "Reports" menu item to Projects
May 22 2019, 12:09 PM
epriestley closed D20495: Add a "Reports" menu item to Projects.
May 22 2019, 12:09 PM
epriestley committed rPf87c1ac362a7: Start the fact daemon in "bin/phd start" (authored by epriestley).
Start the fact daemon in "bin/phd start"
May 22 2019, 11:58 AM
epriestley closed D20489: Start the fact daemon in "bin/phd start".
May 22 2019, 11:58 AM
epriestley committed rP10afe1f2b59c: Fix handling of "null" domain values in Charts (authored by epriestley).
Fix handling of "null" domain values in Charts
May 22 2019, 11:51 AM
epriestley closed D20488: Fix handling of "null" domain values in Charts.
May 22 2019, 11:51 AM
epriestley added a comment to T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion.

See also T13290. Now that full URIs count as mentions (D20527), the text "Differential Revision: ..." in commits is counting as a mention, and "mentioning" the associated revision.

May 22 2019, 11:51 AM · Remarkup, Diffusion
epriestley committed rP146317f2c447: Remove the legacy chart behavior from Maniphest (authored by epriestley).
Remove the legacy chart behavior from Maniphest
May 22 2019, 11:48 AM
epriestley closed D20487: Remove the legacy chart behavior from Maniphest.
May 22 2019, 11:48 AM
epriestley committed rPf8ebc71b8f21: Replace the chart in Maniphest Reports with a chart driven by Facts (authored by epriestley).
Replace the chart in Maniphest Reports with a chart driven by Facts
May 22 2019, 11:44 AM
epriestley closed D20486: Replace the chart in Maniphest Reports with a chart driven by Facts.
May 22 2019, 11:44 AM
epriestley committed rPff6b13872ccf: Add a rough "Chart" Dashboard Panel (authored by epriestley).
Add a rough "Chart" Dashboard Panel
May 22 2019, 11:36 AM
epriestley closed D20485: Add a rough "Chart" Dashboard Panel.
May 22 2019, 11:36 AM
epriestley committed rPc458b50b8569: Render charts from storage instead of just one ad-hoc hard-coded chart (authored by epriestley).
Render charts from storage instead of just one ad-hoc hard-coded chart
May 22 2019, 11:32 AM
epriestley closed D20483: Render charts from storage instead of just one ad-hoc hard-coded chart.
May 22 2019, 11:31 AM
epriestley added a comment to D20503: Automatically select the range for charts in a general way.

(We also don't need to tick the y-max -- we could tick at 100 and draw to 110.)

May 22 2019, 11:30 AM
epriestley added a comment to D20503: Automatically select the range for charts in a general way.

Yeah, I ideally want to try to round up, up to X%, to the nearest "nice" number, like if the range is 0-93 we show 0-100, since "100" is the nearest "nice round number" above 93. There was some simple code for this before (visual-max = actual-max * 1.2 or something) which makes the shape of the graph look good but doesn't produce the most satisfying ticks on the Y-axis or the ideally satisfying Y-max.

May 22 2019, 11:29 AM
joshuaspence updated the diff for D20526: Make it easier to parse external linter versions.

Implement parseVersionOutput in ArcanistExternalLinter so that it can be called from within another instance of ArcanistExternalLinter

May 22 2019, 3:27 AM
joshuaspence updated the summary of D20526: Make it easier to parse external linter versions.
May 22 2019, 3:24 AM
joshuaspence updated the diff for D20542: Be more strict when checking linter versions.

Rebase

May 22 2019, 1:59 AM
joshuaspence requested review of D20542: Be more strict when checking linter versions.
May 22 2019, 1:25 AM
joshuaspence requested review of D20541: Use the checkstyle reporter with JSHint.
May 22 2019, 1:00 AM
joshuaspence updated the diff for D20526: Make it easier to parse external linter versions.

Rebase

May 22 2019, 12:50 AM
joshuaspence added a comment to D20532: Fix handling of empty line/column in linter tests.

Unrelated but I noticed that ArcanistBundleTestCase::testGitRepository fails locally for me when running arc unit --everything:

May 22 2019, 12:37 AM
joshuaspence updated the summary of D20526: Make it easier to parse external linter versions.
May 22 2019, 12:32 AM
epriestley requested review of D20540: Stabilize sorting of feed stories with similar strength.
May 22 2019, 12:05 AM

May 21 2019

amckinley accepted D20506: Stack chart functions in a more physical way.
May 21 2019, 10:56 PM
amckinley accepted D20504: Store charts earlier and build them out a little later.
May 21 2019, 10:40 PM
amckinley accepted D20503: Automatically select the range for charts in a general way.
May 21 2019, 10:30 PM
amckinley accepted D20502: Wrap "<min, max>" chart domain pairs in an "Interval" class.
May 21 2019, 10:27 PM
amckinley accepted D20501: Provide chart function labels over the wire instead of making them up.
May 21 2019, 9:38 PM
amckinley accepted D20500: Label important data on charts.
May 21 2019, 9:36 PM