-
Notifications
You must be signed in to change notification settings - Fork 18
More contributing documentation fixes #1154
Conversation
CONTRIBUTING.md
Outdated
@@ -2,6 +2,8 @@ | |||
|
|||
We're so glad you're thinking about contributing to an 18F open source project! If you're unsure or afraid of anything, just ask or submit the issue or pull request anyways. The worst that can happen is that you'll be politely asked to change something. We appreciate any sort of contribution, and don't want a wall of rules to get in the way of that. | |||
|
|||
*Disclaimer:* Everthing in this guide is not set in stone and subject to change based on changing project requirements and tools. |
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.
Everthing
- > Everything
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.
Made updates to merge
CONTRIBUTING.md
Outdated
@@ -200,7 +202,7 @@ Here are some basic rules to work with action creators successfully: | |||
return Promise.resolve(spaces); | |||
} | |||
``` | |||
- There should not be more then on action with the same data. | |||
- There should not be more then one action with the same data. |
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.
then
-> than
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.
Made updates to merge
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.
just two fixes. looks good though!
@jcscottiii Made the updates. Good for merge. |
No description provided.