-
Notifications
You must be signed in to change notification settings - Fork 113
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
adding resources page #42
Conversation
d5f444b
to
9ecb4f5
Compare
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.
/lgtm
/approve
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.
This looks great! My comments are mostly nits, but I'd like to see this scrubbed for consistent capitalization. I made a few comments, but noticed similar issues in the rest of doc, and didn't want to seem like the Grammar Police by calling out each one :)
9ecb4f5
to
a018e1a
Compare
I had doc nits, but it looks like they're sorted with the new push. |
/lgtm |
a018e1a
to
fd50071
Compare
/lgtm |
#3 and #15