Page MenuHomePhabricator
Feed All Stories

Sep 27 2019

storrgie added a comment to T524: OpenID Authentication as an Auth Option.

Apologies, I should have provided more context. We've been futzing with this project which OIDC. We'd be looking to ensure that it works with Dex.

Sep 27 2019, 1:18 AM · Phabricator
epriestley added a comment to T524: OpenID Authentication as an Auth Option.

I'm not sure how I'd even test OpenID these days. Offhand, I haven't seen any provider offering OpenID in years.

Sep 27 2019, 1:16 AM · Phabricator
storrgie added a comment to T524: OpenID Authentication as an Auth Option.

Would you considered taking prioritized funding for implementing OpenID?

Sep 27 2019, 1:04 AM · Phabricator

Sep 26 2019

epriestley committed rP7a0090f4d070: Fix an issue where the "viewer" is not passed to Bulk Edit controls properly (authored by epriestley).
Fix an issue where the "viewer" is not passed to Bulk Edit controls properly
Sep 26 2019, 7:04 PM
epriestley closed D20841: Fix an issue where the "viewer" is not passed to Bulk Edit controls properly.
Sep 26 2019, 7:03 PM
epriestley requested review of D20841: Fix an issue where the "viewer" is not passed to Bulk Edit controls properly.
Sep 26 2019, 6:49 PM
epriestley committed rP2c06815edb0f: When rendering Jupyter notebook diffs, split code inputs into individual blocks (authored by epriestley).
When rendering Jupyter notebook diffs, split code inputs into individual blocks
Sep 26 2019, 4:05 AM
epriestley closed D20840: When rendering Jupyter notebook diffs, split code inputs into individual blocks.
Sep 26 2019, 4:05 AM
epriestley updated the task description for T11860: Bot email addresses can't be edited.
Sep 26 2019, 4:03 AM · Setup, People
epriestley requested review of D20840: When rendering Jupyter notebook diffs, split code inputs into individual blocks.
Sep 26 2019, 2:53 AM
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20840: When rendering Jupyter notebook diffs, split code inputs into individual blocks.
Sep 26 2019, 2:51 AM · Differential

Sep 25 2019

epriestley committed rP884cd74cc4a5: In prose diffs, use hash-and-diff for coarse "level 0" diffing to scale better (authored by epriestley).
In prose diffs, use hash-and-diff for coarse "level 0" diffing to scale better
Sep 25 2019, 11:51 PM
epriestley closed D20839: In prose diffs, use hash-and-diff for coarse "level 0" diffing to scale better.
Sep 25 2019, 11:51 PM
epriestley closed T13414: Raise the effective corpus size limit in "PhutilProseDifferenceEngine" by using "diff" for coarse passes as Resolved by committing rP884cd74cc4a5: In prose diffs, use hash-and-diff for coarse "level 0" diffing to scale better.
Sep 25 2019, 11:50 PM · Infrastructure
epriestley committed rP9d884f144f01: Add "PhutilProseDiff" classes to "phabricator/" (authored by epriestley).
Add "PhutilProseDiff" classes to "phabricator/"
Sep 25 2019, 11:50 PM
epriestley closed D20838: Add "PhutilProseDiff" classes to "phabricator/".
Sep 25 2019, 11:50 PM
epriestley committed rPHUf39a03df2bc5: Move PhutilProseDiff out of "libphutil/" (authored by epriestley).
Move PhutilProseDiff out of "libphutil/"
Sep 25 2019, 11:47 PM
epriestley closed D20837: Move PhutilProseDiff out of "libphutil/".
Sep 25 2019, 11:47 PM
epriestley committed rP281598d65cfc: Use a hash-and-diff strategy to produce a diff layout for block-based documents (authored by epriestley).
Use a hash-and-diff strategy to produce a diff layout for block-based documents
Sep 25 2019, 11:41 PM
epriestley closed D20836: Use a hash-and-diff strategy to produce a diff layout for block-based documents.
Sep 25 2019, 11:41 PM
epriestley committed rP932d829af339: Improve behavior of inline comment highlight reticle for block diffs (authored by epriestley).
Improve behavior of inline comment highlight reticle for block diffs
Sep 25 2019, 11:39 PM
epriestley closed D20835: Improve behavior of inline comment highlight reticle for block diffs.
Sep 25 2019, 11:39 PM
epriestley committed rPa09b298d85d7: Correct DOM node metadata to let inline comments work against block-based diffs (authored by epriestley).
Correct DOM node metadata to let inline comments work against block-based diffs
Sep 25 2019, 11:38 PM
epriestley closed D20834: Correct DOM node metadata to let inline comments work against block-based diffs.
Sep 25 2019, 11:38 PM
epriestley committed rP1c4450d39f9b: Allow the Jupyter engine to elect to emit diffs, and emit Jupyter documents as… (authored by epriestley).
Allow the Jupyter engine to elect to emit diffs, and emit Jupyter documents as…
Sep 25 2019, 11:32 PM
epriestley closed D20833: Allow the Jupyter engine to elect to emit diffs, and emit Jupyter documents as blocks.
Sep 25 2019, 11:32 PM
epriestley committed rP7ae711ed3e0b: Add a "View as..." option to diff dropdowns for selecting between document… (authored by epriestley).
Add a "View as..." option to diff dropdowns for selecting between document…
Sep 25 2019, 11:29 PM
epriestley closed D20832: Add a "View as..." option to diff dropdowns for selecting between document engines.
Sep 25 2019, 11:29 PM
epriestley committed rPbb71ef6ad6cd: Render image diffs as abstract blocks diffs via DocumentEngine (authored by epriestley).
Render image diffs as abstract blocks diffs via DocumentEngine
Sep 25 2019, 11:25 PM
epriestley closed D20831: Render image diffs as abstract blocks diffs via DocumentEngine.
Sep 25 2019, 11:25 PM
epriestley committed rPa73f592d7d08: Allow DocumentEngine to elect into diff construction (authored by epriestley).
Allow DocumentEngine to elect into diff construction
Sep 25 2019, 11:23 PM
epriestley closed D20830: Allow DocumentEngine to elect into diff construction.
Sep 25 2019, 11:23 PM
epriestley requested review of D20839: In prose diffs, use hash-and-diff for coarse "level 0" diffing to scale better.
Sep 25 2019, 10:14 PM
epriestley added a revision to T13414: Raise the effective corpus size limit in "PhutilProseDifferenceEngine" by using "diff" for coarse passes: D20839: In prose diffs, use hash-and-diff for coarse "level 0" diffing to scale better.
Sep 25 2019, 10:12 PM · Infrastructure
epriestley requested review of D20838: Add "PhutilProseDiff" classes to "phabricator/".
Sep 25 2019, 9:41 PM
epriestley added a revision to T13414: Raise the effective corpus size limit in "PhutilProseDifferenceEngine" by using "diff" for coarse passes: D20838: Add "PhutilProseDiff" classes to "phabricator/".
Sep 25 2019, 9:40 PM · Infrastructure
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20838: Add "PhutilProseDiff" classes to "phabricator/".
Sep 25 2019, 9:40 PM · Differential
epriestley added a revision to T13395: Strip libphutil for parts: D20838: Add "PhutilProseDiff" classes to "phabricator/".
Sep 25 2019, 9:40 PM · libphutil
epriestley requested review of D20837: Move PhutilProseDiff out of "libphutil/".
Sep 25 2019, 9:37 PM
epriestley added a revision to T13395: Strip libphutil for parts: D20837: Move PhutilProseDiff out of "libphutil/".
Sep 25 2019, 9:36 PM · libphutil
epriestley added a revision to T13414: Raise the effective corpus size limit in "PhutilProseDifferenceEngine" by using "diff" for coarse passes: D20837: Move PhutilProseDiff out of "libphutil/".
Sep 25 2019, 9:36 PM · Infrastructure
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20837: Move PhutilProseDiff out of "libphutil/".
Sep 25 2019, 9:36 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.

Screen Shot 2019-09-25 at 1.57.05 PM.png (892×1 px, 149 KB)

Sep 25 2019, 9:18 PM · Differential
epriestley requested review of D20836: Use a hash-and-diff strategy to produce a diff layout for block-based documents.
Sep 25 2019, 9:07 PM
epriestley added a revision to T13414: Raise the effective corpus size limit in "PhutilProseDifferenceEngine" by using "diff" for coarse passes: D20836: Use a hash-and-diff strategy to produce a diff layout for block-based documents.
Sep 25 2019, 9:05 PM · Infrastructure
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20836: Use a hash-and-diff strategy to produce a diff layout for block-based documents.
Sep 25 2019, 9:05 PM · Differential
epriestley requested review of D20835: Improve behavior of inline comment highlight reticle for block diffs.
Sep 25 2019, 8:27 PM
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20835: Improve behavior of inline comment highlight reticle for block diffs.
Sep 25 2019, 8:25 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.

??? it just works ???

Sep 25 2019, 7:50 PM · Differential
epriestley requested review of D20834: Correct DOM node metadata to let inline comments work against block-based diffs.
Sep 25 2019, 7:49 PM
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20834: Correct DOM node metadata to let inline comments work against block-based diffs.
Sep 25 2019, 7:48 PM · Differential
epriestley requested review of D20833: Allow the Jupyter engine to elect to emit diffs, and emit Jupyter documents as blocks.
Sep 25 2019, 7:06 PM
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20833: Allow the Jupyter engine to elect to emit diffs, and emit Jupyter documents as blocks.
Sep 25 2019, 7:04 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.

Jupyter as blocks, no diffing or inlines yet:

Sep 25 2019, 7:03 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.
  • Differential shows a "this file is big, so syntax highlighting is disabled by default" warning even when a document engine which does not use syntax highlighting renders the document.
Sep 25 2019, 6:46 PM · Differential
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20832: Add a "View as..." option to diff dropdowns for selecting between document engines.
Sep 25 2019, 6:21 PM · Differential
epriestley updated the summary of D20832: Add a "View as..." option to diff dropdowns for selecting between document engines.
Sep 25 2019, 6:21 PM
epriestley requested review of D20832: Add a "View as..." option to diff dropdowns for selecting between document engines.
Sep 25 2019, 5:51 PM
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.
  • There's also no "render as native source" option, but there is a "View as Source" option, which doesn't work. Gotcha!
Sep 25 2019, 5:47 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.
  • The options in the "View As..." dropdown are exhaustive, and most do not work, because they aren't based on the changeset being rendered (so we'll give you an option to render a Jupyter notebook as audio, for example). This isn't trivial to fix and it isn't terribly important for this to function as an escape hatch back to old behavior.
  • Since we expect most documents to have a relatively small number of options here, a list of clickable options might be better than a <select /> dropdown.
Sep 25 2019, 5:46 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.

In an effort to "do no harm", I'm planning to add a "Render with Document Engine..." option to the View Options dropdown next. This will let you (for example) view a Jupyter notebook as a raw source diff if you want, if the "fancy" diff is broken or unhelpful for some reason, so you always have an escape hatch back to a lower level representation.

Sep 25 2019, 5:23 PM · Differential
epriestley requested review of D20831: Render image diffs as abstract blocks diffs via DocumentEngine.
Sep 25 2019, 5:16 PM
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20831: Render image diffs as abstract blocks diffs via DocumentEngine.
Sep 25 2019, 5:15 PM · Differential
epriestley added a comment to T13425: Support diffs between abstract block lists in the UI.
  • TwoUp image comments aren't triggering (also in master).
  • OneUp image comments need some cell span adjustments.
Sep 25 2019, 5:10 PM · Differential
epriestley requested review of D20830: Allow DocumentEngine to elect into diff construction.
Sep 25 2019, 4:28 PM
epriestley added a revision to T13425: Support diffs between abstract block lists in the UI: D20830: Allow DocumentEngine to elect into diff construction.
Sep 25 2019, 4:26 PM · Differential
epriestley triaged T13425: Support diffs between abstract block lists in the UI as Normal priority.
Sep 25 2019, 4:19 PM · Differential
epriestley added a comment to T13299: Herald only triggers for commits once, during the initial publish phase.

I've moved the issue in PHI1264 (roughly, "commenting on an unpublished commit should still work normally") here from T13284, since it ended up not really fitting into that scope particularly well.

Sep 25 2019, 4:01 PM · Diffusion, Herald
epriestley updated the task description for T13299: Herald only triggers for commits once, during the initial publish phase.
Sep 25 2019, 4:00 PM · Diffusion, Herald
epriestley updated the task description for T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.
Sep 25 2019, 4:00 PM · Diffusion
epriestley committed rP06edcf270921: Fix an issue where ancestors of permanent refs might not be published during… (authored by epriestley).
Fix an issue where ancestors of permanent refs might not be published during…
Sep 25 2019, 3:55 PM
epriestley closed D20829: Fix an issue where ancestors of permanent refs might not be published during import or if a branch is later made permanent.
Sep 25 2019, 3:55 PM
epriestley closed T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published as Resolved by committing rP06edcf270921: Fix an issue where ancestors of permanent refs might not be published during….
Sep 25 2019, 3:55 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

This fix is not retroactive (and, I think, isn't easy to make retroactive in an obviously safe way) so you'll need to manually repair affected repositories if you want to fix existing commits affected by this bug. This bug doesn't likely doesn't have any far-reaching implications so there's no need to go on a grand adventure to hunt these down, but if you've run into some you can fix them like this:

Sep 25 2019, 3:53 PM · Diffusion
epriestley requested review of D20829: Fix an issue where ancestors of permanent refs might not be published during import or if a branch is later made permanent.
Sep 25 2019, 3:49 PM
epriestley added a revision to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published: D20829: Fix an issue where ancestors of permanent refs might not be published during import or if a branch is later made permanent.
Sep 25 2019, 3:48 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

Add a bin/repository refs --rebuild or similar flag to repair repositories affected by this bug.

Sep 25 2019, 3:43 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

If this theory holds up, the "swap a branch" case can be reproduced like this:

Sep 25 2019, 3:39 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

Retrying this on the patch:

Sep 25 2019, 3:19 PM · Diffusion
epriestley added a comment to T13424: Support builds with TeamCity in Harbormaster.

The problem isn't that it's in rough shape (I'm fine with bringing rough stuff upstream), but that it's something I may eventually want to license as a paid extension. I generally want to stop bringing "free glue for paid systems" upstream (T13229).

Sep 25 2019, 2:22 PM · Harbormaster
swisspol added a comment to T13424: Support builds with TeamCity in Harbormaster.

How about landing this as a prototype?

Sep 25 2019, 4:46 AM · Harbormaster
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

The "import a repo" case can be reproduced like this:

Sep 25 2019, 1:50 AM · Diffusion

Sep 24 2019

epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

If this theory holds up, the "swap a branch" case can be reproduced like this:

Sep 24 2019, 11:57 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

If this theory holds up, the "swap a branch" case can be reproduced like this:

Sep 24 2019, 11:17 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

The algorithm in PhabricatorRepositoryRefEngine->updateRefs() is approximately:

Sep 24 2019, 10:49 PM · Diffusion
epriestley added a comment to T13284: When a previously non-permanent ref is marked permanent, unpublished commits reachable from that ref are not published.

See PHI1447. In this case, the install reports a sequence of events roughly like this:

Sep 24 2019, 10:09 PM · Diffusion
epriestley lowered the priority of T13413: Chrome 77 rendering engine freezes when elements are nested with particular word-wrap / white-space CSS behavior from Normal to Wishlist.

This now appears to be fixed in the release version of Chrome. We can remove the workaround in some future change.

Sep 24 2019, 6:10 PM · Google Chrome
epriestley committed rP6af776f84a66: Allow installs to provide "Request a Username Change" instructions (authored by epriestley).
Allow installs to provide "Request a Username Change" instructions
Sep 24 2019, 6:09 PM
epriestley closed D20828: Allow installs to provide "Request a Username Change" instructions.
Sep 24 2019, 6:09 PM
epriestley closed T13420: Update "Change Username" to make it more friendly for non-administrators as Resolved by committing rP6af776f84a66: Allow installs to provide "Request a Username Change" instructions.
Sep 24 2019, 6:09 PM · People, Auth
epriestley committed rP0c331458a88b: When non-administrators click "Change Username", explain why they can't continue (authored by epriestley).
When non-administrators click "Change Username", explain why they can't continue
Sep 24 2019, 6:09 PM
epriestley closed D20827: When non-administrators click "Change Username", explain why they can't continue.
Sep 24 2019, 6:08 PM
epriestley committed rP058b7ae9450d: Update "Change Username" instructions to be less foreboding (authored by epriestley).
Update "Change Username" instructions to be less foreboding
Sep 24 2019, 6:08 PM
epriestley closed D20826: Update "Change Username" instructions to be less foreboding.
Sep 24 2019, 6:08 PM
epriestley committed rPb1d4d5c00ce1: Add an "{anchor #xyz}" rule to Remarkup (authored by epriestley).
Add an "{anchor #xyz}" rule to Remarkup
Sep 24 2019, 6:04 PM
epriestley closed T4280: Embed anchor in remark up text and later link to it. as Resolved by committing rPb1d4d5c00ce1: Add an "{anchor #xyz}" rule to Remarkup.
Sep 24 2019, 6:04 PM · Remarkup
epriestley closed T4280: Embed anchor in remark up text and later link to it. , a subtask of T13410: Update Phriction UI structural behaviors, as Resolved.
Sep 24 2019, 6:04 PM · Phriction
epriestley closed D20825: Add an "{anchor #xyz}" rule to Remarkup.
Sep 24 2019, 6:04 PM
epriestley committed rPbff72ce3b582: Generate more friendly anchor names for header sections in Remarkup (authored by epriestley).
Generate more friendly anchor names for header sections in Remarkup
Sep 24 2019, 6:03 PM
epriestley closed D20821: Generate more friendly anchor names for header sections in Remarkup.
Sep 24 2019, 6:03 PM