Page MenuHomePhorge

Update arcanist readme to reference Phorge
ClosedPublic

Authored by Matthew on Jun 20 2021, 01:05.

Details

Reviewers
20after4
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Maniphest Tasks
T15006: Re-brand Phorge
Test Plan

Looked at the rendered markdown.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jun 20 2021, 01:05
Matthew requested review of this revision.Jun 20 2021, 01:05

Fix a link that will break once we regenerate the documentation.

eax added inline comments.
README.md
1–2

This line as well?

20after4 added a subscriber: 20after4.

Shouldn't we also think about changing the name of arcanist or does it make sense to have a fork with the same name?

This revision is now accepted and ready to land.Jun 20 2021, 12:24
README.md
6

How about changing the protocol to https?

Shouldn't we also think about changing the name of arcanist or does it make sense to have a fork with the same name?

Not sure the answer to that one... Would you like to open a task to discuss?

Address code review comments

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