-
Notifications
You must be signed in to change notification settings - Fork 76
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
Add opensource & complete projects to React guide #34
base: master
Are you sure you want to change the base?
Conversation
Will wait for @M-ZubairAhmed to review and merge this |
@@ -41,6 +41,9 @@ | |||
* [React without ES6](#react-without-es6) | |||
* [Reconcilation - Diff Algorithm](#reconcilation---diff-algorithm) | |||
|
|||
* [Complete Projects](#complete-projects-that-you-can-have-a-look) |
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.
I dont think it is complete so lets remove the title complete projects
and let it be just open source projects you can contribute to
@@ -41,6 +41,9 @@ | |||
* [React without ES6](#react-without-es6) | |||
* [Reconcilation - Diff Algorithm](#reconcilation---diff-algorithm) | |||
|
|||
* [Complete Projects](#complete-projects-that-you-can-have-a-look) | |||
* [Opensource projects you can contribute to](#opensource-projects-you-can-contribute-to) |
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.
Did you generate table of content with doc-toc
or did you insert manually?
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.
Manually
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.
Shall we add doc-toc as a git hook?
https://github.com/thlorenz/doctoc#usage-as-a-git-hook
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.
yes please add
What are we to do with this PR? |
please resolve this, so we can proceed to merge |
For #29