Skip to content

Conversation

@lencioni
Copy link
Contributor

While I was poking around in this file today, I noticed a few very minor things that could use some attention. These commits mostly just adjust some of the formatting--some for the benefit of those reading the rendered markdown, and others for the benefit of those looking at the markdown source. There's likely still more that could be done, but I figured I'd throw this up here since I think it makes a few small improvements.

When I added this section in f2dc504 I didn't do a very good job of
following the style used in other parts of this repo. This commit
remedies that.
@ljharb
Copy link
Collaborator

ljharb commented Dec 23, 2015

This all seems good to me

I noticed a number of places in this document where code was being
referenced but it was not marked as such. Adding backticks will instruct
the markdown parser to format these bits as code, which should improve
the readability of this document.
I noticed that this heading wasn't listed in the table of contents.
Adding it will help people see at-a-glance what they might find in this
document.
I noticed that some newlines and indentation were a little inconsistent
in this file, so I decided to smooth things out a bit. This should help
people who decide to modify this document in the future. When
determining which is the "right" way to format these things, I decided
to use the style used by base readme as a guide.
@lencioni lencioni force-pushed the react-readme-cleanup branch from 2143857 to 474087c Compare December 23, 2015 18:12
ljharb added a commit that referenced this pull request Dec 23, 2015
@ljharb ljharb merged commit 1ac8baa into airbnb:master Dec 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants