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
Branch
master
Lint
Lint Warnings
Unit
No Test Coverage
Build Status
Buildable 18
Build 18: arc lint + arc unit

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.