Page MenuHomePhabricator
Differential D11542

Add a more complete README for GitHub
ClosedPublic
Actions

Authored by chad on Jan 28 2015, 8:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 3, 5:28 PM
Unknown Object (File)
Sat, Dec 28, 2:12 PM
Unknown Object (File)
Fri, Dec 27, 10:24 PM
Unknown Object (File)
Fri, Dec 27, 9:07 PM
Unknown Object (File)
Mon, Dec 23, 2:03 PM
Unknown Object (File)
Mon, Dec 23, 12:38 AM
Unknown Object (File)
Sun, Dec 22, 10:31 PM
Unknown Object (File)
Sun, Dec 22, 7:13 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP4d6aea7ce81a: Add a more complete README for GitHub
Summary

Not sure this is entirely correct or needed as a separate file, but I think GitHub's README could use some additonal information

Test Plan

Used a Markdown editor

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 27776.Jan 28 2015, 8:36 PM
chad retitled this revision from to Add a more complete README for GitHub.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 28 2015, 8:36 PM
epriestley edited edge metadata.Jan 28 2015, 8:42 PM

Isn't this exactly the same as the existing README?

chad added a comment.Jan 28 2015, 8:42 PM

haha wtf.

epriestley added a comment.Jan 28 2015, 8:43 PM

I mean, it's a beautiful document, I'm just not sure we need two of them.

chad updated this revision to Diff 27780.Jan 28 2015, 8:43 PM
chad edited edge metadata.
  • derpy derpy
epriestley added a comment.Jan 28 2015, 8:44 PM

Let's just remove README? We should parse all this stuff pretty much the same way in Remarkup nowadays.

chad added a comment.Jan 28 2015, 8:44 PM

My main question is, do we need two documents? Can these just be one, somehow?

chad added a comment.Jan 28 2015, 8:45 PM

Ok, that's what I thought just wasn't sure.

chad updated this revision to Diff 27782.Jan 28 2015, 8:47 PM
  • Remove README
epriestley accepted this revision.Jan 28 2015, 8:48 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 28 2015, 8:48 PM
Closed by commit rP4d6aea7ce81a: Add a more complete README for GitHub (authored by chad, committed by chad). · Explain WhyJan 28 2015, 8:49 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
31 lines

Diff 27784

View Options

README

Loading...
View Options

README.md

Loading...
Phabricator · Built by Phacility