Skip to content

Fix documentation of the release process#12337

Merged
gaearon merged 3 commits intofacebook:masterfrom
delftswa2018:fix-doc-script
Apr 16, 2018
Merged

Fix documentation of the release process#12337
gaearon merged 3 commits intofacebook:masterfrom
delftswa2018:fix-doc-script

Conversation

@FlXlT
Copy link
Contributor

@FlXlT FlXlT commented Mar 6, 2018

Adjusted some grammar in the scripts/release/README.md file.

  • Adjusted title to infer that there are different kinds of release scripts
  • The first sentence infers that there is one release script. However, it seems that the release process includes two scripts rather than being a script on its own.
  • Officially, and also confirmed on the homepage readme, the npm logo and name are with lowercase letters.
  • Some abbreviations were missing punctuation.

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.

5 participants