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

3 participants