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

	F14718467: D11542.diff
	Fri, Jan 17, 8:28 PM

	F14715921: D11542.diff
	Fri, Jan 17, 6:26 PM

	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

Subscribers

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

Branch

markdown-readme

Lint

Lint Warnings

Severity	Location	Code	Message
Warning	README.md:1	TXT3	Line Too Long
Warning	README.md:14	TXT3	Line Too Long
Warning	README.md:16	TXT3	Line Too Long
Warning	README.md:19	TXT3	Line Too Long
Warning	README.md:22	TXT3	Line Too Long
Warning	README.md:25	TXT3	Line Too Long
Warning	README.md:28	TXT3	Line Too Long

Unit

No Test Coverage

Build Status

Buildable 4129
Build 4142: [Placeholder Plan] Wait for 30 Seconds

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

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

haha wtf.

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

derpy derpy

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

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

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

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

Path

Size

31 lines

Diff 27782

README

Loading...

README.md

Loading...