Uh oh!
There was an error while loading. Please reload this page.
documentation: added License to the README.md - #4294
Merged
Conversation
Timer
commented
Apr 15, 2018
Contributor
I appreciate the PR, but I'm not sure if this is necessary -- a LICENSE file is pretty standard and GitHub even specifies the license up in the project bar (above Clone or download). Thanks! |
gaearon
commented
Apr 15, 2018
Contributor
Hmm. Actually I’d like to have it in README. But the wording is a bit too colloquial. I’d prefer just “Create React App is open source software licensed as MIT.” |
gaearon
commented
Apr 15, 2018
Contributor
(The reason I’d like to have it is to bring extra attention to it not being licensed as BSD+P like in the past) |
thiagopaiva99
commented
Apr 16, 2018
ContributorAuthor
Hey, thanks for reopening the PR! I have changed the way the license is described as @gaearon suggested! Thank you! |
Closed
Timer
commented
Apr 21, 2018
Contributor
Thanks! |
kellyrmilligan added a commit
to kellyrmilligan/create-react-app
that referenced
this pull request
May 2, 2018
* upstream/next: (35 commits) Update envinfo and issue template (react#4375) Update sass-loader to 7.0.1 (react#4376) Support package distribution tags (react#4350) fix broken css module support in prod (react#4361) Bumped jest version to 22.4.1 (react#4362) bump babel 7 to beta 46 bump lint-staged to node 10 compatible version documentation: Added License to the README.md (react#4294) Bump `fsevents`. (react#4331) Fix typo in e2e-simple.sh comment (react#4323) Add Sass loader (react#4195) Fix some typos in README.md (react#4286) Added learnstorybook.com to Storybook links (react#4298) Document multiple build environments via `env-cmd` react#4071 (react#4117) Fixed link to CSS imports blog post Update CSS Modules localIndetName (react#4192) Enable loose mode for `class-properties` (react#4248) bump babel 7 beta (react#4253) Small typo fixreact#4217 Changelog for 1.1.4 ...
kellyrmilligan added a commit
to kellyrmilligan/create-react-app
that referenced
this pull request
May 2, 2018
* next: (35 commits) Update envinfo and issue template (react#4375) Update sass-loader to 7.0.1 (react#4376) Support package distribution tags (react#4350) fix broken css module support in prod (react#4361) Bumped jest version to 22.4.1 (react#4362) bump babel 7 to beta 46 bump lint-staged to node 10 compatible version documentation: Added License to the README.md (react#4294) Bump `fsevents`. (react#4331) Fix typo in e2e-simple.sh comment (react#4323) Add Sass loader (react#4195) Fix some typos in README.md (react#4286) Added learnstorybook.com to Storybook links (react#4298) Document multiple build environments via `env-cmd` react#4071 (react#4117) Fixed link to CSS imports blog post Update CSS Modules localIndetName (react#4192) Enable loose mode for `class-properties` (react#4248) bump babel 7 beta (react#4253) Small typo fixreact#4217 Changelog for 1.1.4 ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What:
Added License in the README
Why:
Keep the docs consistent and informing for everyone the project License in a easiest way.
How:
[x] adding License section on README
Check:
[x] Documentation
[ ] Tests
[x] Ready to be merged