You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+5-5Lines changed: 5 additions & 5 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -25,12 +25,12 @@ There are three important things to know:
25
25
26
26
This a checklist of things to keep in your mind when opening pull requests for this project.
27
27
28
-
-1. Before pushing anything, validate your pull request with `go test`
29
-
0. Make sure you have signed the [Contributor License Agreement](#contributor-license-agreement)
30
-
1. Make sure any added dependency is licensed under Apache v2.0 license
31
-
2. Strive for very high unit-test coverage and favor testing productive code over mocks
28
+
0. Before pushing anything, validate your pull request with `go test`
29
+
1. Make sure you have signed the [Contributor License Agreement](#contributor-license-agreement)
30
+
2. Make sure any added dependency is licensed under Apache v2.0 license
31
+
3. Strive for very high unit-test coverage and favor testing productive code over mocks
32
32
(mock in depth wherever possible)
33
-
3. Update the README.md with details of changes to the options
33
+
4. Update the README.md with details of changes to the options
34
34
35
35
Pull requests will be tested and validated by maintainers. In case small changes are needed (e.g., correcting typos), the maintainers may fix those issues themselves.
36
36
In case of larger issues, you may be asked to apply modifications to your changes before the Pull Request can be merged.
0 commit comments