-
-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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 Neuroscience #962
Add Neuroscience #962
Conversation
Not followed:
Suggestions:
|
@legodude17 Thanks for the pointers. Made the changes. |
readme.md
Outdated
@@ -442,6 +442,7 @@ | |||
|
|||
## Miscellaneous | |||
|
|||
- [Neuroscience](https://github.com/analyticalmonk/awesome-neuroscience) |
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 include a description.
904f6b6
to
b644485
Compare
@legodude17 Added a description. |
@sindresorhus Any comments? This looks fine to me |
readme.md
Outdated
@@ -442,6 +442,7 @@ | |||
|
|||
## Miscellaneous | |||
|
|||
- [Neuroscience](https://github.com/analyticalmonk/awesome-neuroscience) - Study of the nervous system and brain |
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.
The entry in the Awesome list should be added at the bottom of the appropriate category.
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.
And missing a dot at the end.
@sindresorhus Made the changes. |
LGTM, although I would've wanted to see more links. There hasn't been that much additions since the list started. |
Looks pretty good. Just some minor nitpick left. Since you have the nice header graphics, you don't really need the text title. You can place the Awesome badge somewhere else, for example, under the intro.
|
@sindresorhus Made the changes. |
Great. One last thing, you need to set the width of the header image to half its actual size (That means using an |
Added :) |
[https://github.com/analyticalmonk/awesome-neuroscience]
[This is a curated list of awesome neuroscience libraries, software and any content related to the domain. The popularity of machine learning, neural networks and related technologies has led to an increased interest in study of the human brain. Neuroscience is a fascinating subject and this list will help enthusiasts and practitioners to explore its computational aspects.]
By submitting this pull request I confirm I've read and complied with the below requirements.
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
For example,
Add Name of List
, notUpdate readme.md
orAdd awesome list
.Example:
- [Fish](…) - User-friendly shell.
, not- [Fish](…) - Resources for Fish.
.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
awesome-list
&awesome
as GitHub topics. I encourage you to add more relevant topics.Contents
, notTable of Contents
.contributing.md
. Casing is up to you.Example:
- [AVA](…) - JavaScript test runner.
A
/An
prefixes in the descriptions.Node.js
, notNodeJS
ornode.js
.You can still use Travis for list linting, but the badge has no value in the readme.