Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Update contributing guide #157

Merged
merged 2 commits into from Oct 18, 2017
Merged

Update contributing guide #157

merged 2 commits into from Oct 18, 2017

Conversation

wtrocki
Copy link
Member

@wtrocki wtrocki commented Oct 18, 2017

Motivation

Update contributing guide.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 569c1b8 on contributing-update into ** on master**.

1 similar comment
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 569c1b8 on contributing-update into ** on master**.

Copy link
Contributor

@JameelB JameelB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small fixes :)

CONTRIBUTING.md Outdated
@@ -8,11 +8,19 @@ For more information about the FeedHenry project, visit
## Asking for help

Whether you're contributing a new feature or bug fix, or simply submitting a
ticket, the FeedHenry team is available for technical advice or feedback.
You can reach us at #feedhenry and #feedhenry-raincatcher on [Freenode IRC](https://freenode.net/) or the
ticket, the RainCatcheer team is available for technical advice or feedback.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo Raincatcher

CONTRIBUTING.md Outdated
## Best practices

- Bugfixes should contain only changes that are related to the purpose of the bug
- Description should contain explaination for proposed changes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.. should contain an explanation for the proposed changes

CONTRIBUTING.md Outdated

- Bugfixes should contain only changes that are related to the purpose of the bug
- Description should contain explaination for proposed changes
- It's recomended to consult feature requests with the team before starting implementation
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

recommended

CONTRIBUTING.md Outdated
- Bugfixes should contain only changes that are related to the purpose of the bug
- Description should contain explaination for proposed changes
- It's recomended to consult feature requests with the team before starting implementation
- Send us message on chat or email if you need assistance with any work.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Send us a message

CONTRIBUTING.md Outdated

## Best practices

- Bugfixes should contain only changes that are related to the purpose of the bug
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should only contain

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6fcd73e on contributing-update into ** on master**.

1 similar comment
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6fcd73e on contributing-update into ** on master**.

@wtrocki wtrocki merged commit 38caf6b into master Oct 18, 2017
@wtrocki wtrocki deleted the contributing-update branch October 18, 2017 15:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants