Skip to content

Conversation

@lencioni
Copy link
Contributor

@lencioni lencioni commented Feb 4, 2016

Adjusted formatting, removed old copy, sorted things, and added missing transforms.

I found this wall of text a little difficult to parse. I think it works
much better with some headings. While I was at it, I decided to wrap
some long lines to be more consistent with the rest of this document.
The alphabetical sorting is easy to understand, making it easier for
people to find scripts that they are looking for and making it easier
for future devs to know where to add new scripts.
This mentions "These three scripts" when there are four mentioned, and
I am unable to find any references to `--no-explicit-require` in code,
so I think this sentence is outdated and should be removed. Digging
through history, it seems like this was brought over from 1c672cb
when code was imported from the React repo.
This useful transform was undocumented, so I decided to copy the comment
from the top of the file into the readme. This will help make it more
discoverable.
This transform exists, but was not mentioned in the readme. I haven't
taken the time to look at it and understand what it does, so I didn't
write a description, but I figure that something is better than nothing.
This will give us a little bit of syntax highlighting and cut down on
the number of bullet points on the page. My goal here is to make this
document easier to read.
@cpojer
Copy link
Member

cpojer commented Feb 4, 2016

Swweeet!

cpojer added a commit that referenced this pull request Feb 4, 2016
@cpojer cpojer merged commit cc2ce14 into reactjs:master Feb 4, 2016
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.

3 participants