Skip to content

Commit 9371a92

Browse files
authored
Update CONTRIBUTING.md
1 parent e8b030a commit 9371a92

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

CONTRIBUTING.md

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ Pull Requests
4949

5050
Pull requests are always invited. However, we ask that you follow these guidelines:
5151

52+
- It is wiser to discuss your ideas first as part of an issue before you start coding. If you omit this step and code first, be prepare to have your code receive scrutiny and be dropped.
5253
- Users should provide a rationale for their changes. Does it improve performance? Does it add a feature? Does it improve maintainability? Does fix a bug? This must be explicitly stated as part of the pull request. Do not propose changes based on taste or intuition. We do not delegate programming to tools: that some tool suggested a code change is not reason enough to change the code.
5354
1. When your code improves performance, please document the gains with a benchmark using hard numbers.
5455
2. If your code fixes a bug, please be either fix a failing test, or propose a new test.
@@ -57,11 +58,21 @@ Pull requests are always invited. However, we ask that you follow these guidelin
5758
- New features must be accompanied of new tests, in general.
5859
- Your code should pass our continuous-integration tests. It is your responsability to ensure that your proposal pass the tests. We do not merge pull requests that would break our build.
5960

61+
If the benefits of your proposed code remain unclear, we may choose to discard your code: that is not an insult, we frequently discard our own code. We may also consider various alternatives and choose another path. Again, that is not an insult or a sign that you have wasted your time.
62+
6063
Code of Conduct
6164
---------------
6265

6366
Though we do not have a formal code of conduct, we will not tolerate bullying, bigotry or
64-
intimidation. Everyone is welcome to contribute.
67+
intimidation. Everyone is welcome to contribute. If you have concerns, you can raise them privately with the core team members (e.g., D. Lemire, J. Keiser).
68+
69+
We welcome contributions from women and less represented groups. If you need help, please reach out.
70+
71+
Consider the following points when engaging with the project:
72+
73+
- We discourage arguments from authority: ideas are discusssed on their own merits and not based on who stated it.
74+
- Be mindful that what you may view as an aggression is maybe merely a difference of opinion or a misunderstanding.
75+
- Be mindful that a collection of small aggressions, even if mild in isolation, can become harmful.
6576

6677
Getting Started Hacking
6778
-----------------------

0 commit comments

Comments
 (0)