Page MenuHomePhabricator

Remove push to IRC from "readme.md" too
ClosedPublic

Authored by epriestley on Oct 24 2015, 5:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 12:21 PM
Unknown Object (File)
Wed, Apr 10, 3:47 AM
Unknown Object (File)
Tue, Apr 9, 3:03 AM
Unknown Object (File)
Mon, Apr 8, 3:37 PM
Unknown Object (File)
Fri, Apr 5, 11:31 PM
Unknown Object (File)
Mar 4 2024, 6:43 PM
Unknown Object (File)
Mar 4 2024, 6:43 PM
Unknown Object (File)
Mar 4 2024, 6:43 PM
Subscribers
None

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP1a4e5931f86f: Remove push to IRC from "readme.md" too
Summary

Brings this in line with other support docs and pushes users toward the modern stuff.

Test Plan

Read documentation.

Diff Detail

Repository
rP Phabricator
Branch
supportx
Lint
Lint Warnings
SeverityLocationCodeMessage
WarningREADME.md:1TXT3Line Too Long
WarningREADME.md:23TXT3Line Too Long
WarningREADME.md:27TXT3Line Too Long
Unit
No Test Coverage
Build Status
Buildable 8399
Build 9648: Run Core Tests
Build 9647: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Remove push to IRC from "readme.md" too.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
README.md
1

I dropped "open source" here since you only ever read this document on GitHub when looking at the repository anyway (so it's obviously open source) and I think "open source" is the least important part of our elevator pitch, and in some phrasings it gets confusing whether, e.g., "open source software development platform" means we're for open source or are open source.

chad edited edge metadata.
This revision is now accepted and ready to land.Oct 25 2015, 12:38 AM
This revision was automatically updated to reflect the committed changes.