Skip to content

📝 Add contributing documentation#7

Open
Whisper40 wants to merge 1 commit into
masterfrom
doc
Open

📝 Add contributing documentation#7
Whisper40 wants to merge 1 commit into
masterfrom
doc

Conversation

@Whisper40
Copy link
Copy Markdown
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
Copy Markdown

@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.

Comment thread CONTRIBUTING.md

#### 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.
Copy link
Copy Markdown

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)

Comment thread CONTRIBUTING.md

### 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.
Copy link
Copy Markdown

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