Skip to content
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 contributing documentation #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

📝 Add contributing documentation #7

wants to merge 1 commit into from

Conversation

Whisper40
Copy link
Contributor


Informe que le contributeur peut créer une issue et linkée une PR apportant le correctif/fonctionnalité. Permet de savoir comment créer une nouvelle version de l'outil.

---
Informe que le contributeur peut créer une issue et linkée une PR apportant le correctif/fonctionnalité.
Permet de savoir comment créer une nouvelle version de l'outil.
Copy link

@evrardjp evrardjp left a comment

Choose a reason for hiding this comment

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

A few nits. It's a welcomed improvement.


#### Solve an issue

Scan through our existing issues to find one that interests you. As a general rule, we don’t assign issues to anyone. If you find an issue to work on, you are welcome to open a PR with a fix.

Choose a reason for hiding this comment

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

"If you find an issue to work on, you are welcome to open a PR with a fix."

You can remove the first sentence, and on top of it, there is a typo on the " ' " sign. (Incorrect character)


### Make Changes

If you'd like to add a feature or make a fix, please submit a PR explaining how your MR meets the need.

Choose a reason for hiding this comment

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

Maybe we should explain how to test changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants