Page MenuHomePhorge

Update arcanist readme to reference Phorge
ClosedPublic

Authored by Matthew on Jun 20 2021, 01:05.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 1, 06:33
Unknown Object (File)
Sat, Oct 1, 06:33
Unknown Object (File)
Sat, Oct 1, 06:32
Unknown Object (File)
Sat, Oct 1, 06:32
Unknown Object (File)
Sat, Oct 1, 06:32
Unknown Object (File)
Wed, Sep 28, 21:28
Unknown Object (File)
Wed, Sep 28, 13:17
Unknown Object (File)
Tue, Sep 27, 01:51

Details

Reviewers
20after4
Group Reviewers
O1: Blessed Committers
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

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.