Page MenuHomePhorge

Update Diviner documentation to reference Phorge instead of Phabricator.
ClosedPublic

Authored by Matthew on Jun 19 2021, 23:21.

Details

Summary

This commit also removes references to support pacts and updates links to point to the new upstream.

Test Plan

Generated Diviner documentation on a local install and verified that the changes look good.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Matthew requested review of this revision.Jun 19 2021, 23:21
chris subscribed.

Couple tiny nits and otherwise LGTM!

src/docs/contributor/adding_new_classes.diviner
149–150

Remove the libphutil reference while we're in here?

src/docs/contributor/bug_reports.diviner
129

Maybe omit this until we set this up to avoid confusion?

src/docs/contributor/database.diviner
200

Is there a task tracking this yet? Looks like an edit at src/applications/config/option/PhabricatorMySQLConfigOptions.php:38, just want to make sure we don't forget down the road

src/docs/contributor/reproduction_steps.diviner
105

ref previous comment

src/docs/user/configuration/configuration_locked.diviner
60
137
src/docs/user/userguide/diffusion_hosting.diviner
353

Should probably make a note elsewhere to rename the files in resources/sshd/ so this doesn't get overlooked later

This revision is now accepted and ready to land.Jun 19 2021, 23:58
Matthew added inline comments.
src/docs/contributor/database.diviner
200

T15016

I've also reverted this change in the meantime.

src/docs/user/userguide/diffusion_hosting.diviner
353
Matthew marked 2 inline comments as done.

Make changes based on code review comments.

This revision was landed with ongoing or failed builds.Jun 20 2021, 00:54
This revision was automatically updated to reflect the committed changes.