Update contributing.md (#4275)
* Update contributing.md * Update contributing.md updated as per comments about "closes" and "fixes" special words * update contributing.md as per suggestions Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu> Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu>
This commit is contained in:
parent
08bc7949fc
commit
f096c35234
@ -61,7 +61,7 @@ This describes the overall steps to your first code contribution in OpenRefine.
|
|||||||
|
|
||||||
- Make changes to the code to fix the issue. If you are changing backend code, it would be great if you could also write a test in Java to demonstrate the fix. You can imitate existing tests for that. We currently do not have frontend tests.
|
- Make changes to the code to fix the issue. If you are changing backend code, it would be great if you could also write a test in Java to demonstrate the fix. You can imitate existing tests for that. We currently do not have frontend tests.
|
||||||
|
|
||||||
- commit your changes, using a message that contains "closes #1234" or "fixes #1234", this will link the commit to the issue you are working on.
|
- commit your changes, using a message that contains one of the special words "closes" and "fixes" which are detected by Github, followed by the issue number, e.g. "closes #1234" or "fixes #1234", this will link the commit to the issue you are working on.
|
||||||
|
|
||||||
- push your branch to your fork and create a pull request for it, explaining the approach you have used, any design decisions you have made.
|
- push your branch to your fork and create a pull request for it, explaining the approach you have used, any design decisions you have made.
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user