-
-
Notifications
You must be signed in to change notification settings - Fork 27k
Removed errors #7895
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removed errors #7895
Conversation
create react-app >>> create-react-app Sourcemaps >>> source maps webkit >>>> Web kit
Hi Priyanshu2001! Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. If you have received this in error or have any questions, please contact us at [email protected]. Thanks! |
Thanks for your submission! I can see the confusion but its actually correct, check https://yarnpkg.com/lang/en/docs/cli/create/ Can you revert that one and keep the other typo fixes? Thanks! |
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Priyanshu2001 - Can you update the PR with suggested changes? Thanks!
Okk!!
I have done it sir.
Hope you will merge now.
Thank you!
…On Wed, Nov 6, 2019, 14:01 Andreas Cederström ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Hi @Priyanshu2001 <https://github.com/Priyanshu2001> - Can you update the
PR with suggested changes? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#7895?email_source=notifications&email_token=ANMCBBNVE2YI742L2T2IJNDQSJ6HVA5CNFSM4JGGA62KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCKOLRIQ#pullrequestreview-312260770>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANMCBBIRIZZW44FB4ZKM5HLQSJ6HVANCNFSM4JGGA62A>
.
|
@@ -145,10 +145,10 @@ Your environment will have everything you need to build a modern single-page Rea | |||
|
|||
- React, JSX, ES6, TypeScript and Flow syntax support. | |||
- Language extras beyond ES6 like the object spread operator. | |||
- Autoprefixed CSS, so you don’t need `-webkit-` or other prefixes. | |||
- Autoprefixed CSS, so you don’t need `-WebKit-` or other prefixes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
- A fast interactive unit test runner with built-in support for coverage reporting. | ||
- A live development server that warns about common mistakes. | ||
- A build script to bundle JS, CSS, and images for production, with hashes and sourcemaps. | ||
- A build script to bundle JS, CSS, and images for production, with hashes and source maps. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
README.md
Outdated
|
||
## React Native | ||
## React Native :- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this
README.md
Outdated
@@ -199,6 +199,6 @@ We are grateful to the authors of existing related projects for their ideas and | |||
- [@insin](https://github.com/insin) | |||
- [@mxstbr](https://github.com/mxstbr) | |||
|
|||
## License | |||
## License :- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this, thanks
Done sir!
…On Wed, Nov 6, 2019, 19:35 Andreas Cederström ***@***.***> wrote:
***@***.**** requested changes on this pull request.
------------------------------
In README.md
<#7895 (comment)>
:
> @@ -145,10 +145,10 @@ Your environment will have everything you need to build a modern single-page Rea
- React, JSX, ES6, TypeScript and Flow syntax support.
- Language extras beyond ES6 like the object spread operator.
-- Autoprefixed CSS, so you don’t need `-webkit-` or other prefixes.
+- Autoprefixed CSS, so you don’t need `-WebKit-` or other prefixes.
Nice catch!
------------------------------
In README.md
<#7895 (comment)>
:
> - A fast interactive unit test runner with built-in support for coverage reporting.
- A live development server that warns about common mistakes.
-- A build script to bundle JS, CSS, and images for production, with hashes and sourcemaps.
+- A build script to bundle JS, CSS, and images for production, with hashes and source maps.
Nice catch!
------------------------------
In README.md
<#7895 (comment)>
:
>
-## React Native
+## React Native :-
Please revert this
------------------------------
In README.md
<#7895 (comment)>
:
> @@ -199,6 +199,6 @@ We are grateful to the authors of existing related projects for their ideas and
- ***@***.***(https://github.com/insin)
- ***@***.***(https://github.com/mxstbr)
-## License
+## License :-
Please revert this, thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#7895?email_source=notifications&email_token=ANMCBBO3GF7QRODFQYPM74LQSLFKHA5CNFSM4JGGA62KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCKP2AMA#pullrequestreview-312451120>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANMCBBMLDONWYKWUHQW43WDQSLFKHANCNFSM4JGGA62A>
.
|
create react-app >>> create-react-app
Sourcemaps >>> source maps
webkit >>>> Web kit