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
+2-2Lines changed: 2 additions & 2 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -61,11 +61,11 @@ Making Changes
61
61
62
62
+ Create a _topic branch_ for your isolated work.
63
63
* Usually you should base your branch from the `master` branch.
64
-
* A good topic branch name can be the JIRA bug ID plus a keyword, for example,`COMPRESS-123-InputStream`.
64
+
* A good topic branch name can be the JIRA bug ID plus a keyword, e.g.`COMPRESS-123-InputStream`.
65
65
* If you have submitted multiple JIRA issues, try to maintain separate branches and pull requests.
66
66
+ Make commits of logical units.
67
67
* Make sure your commit messages are meaningful and in the proper format. Your commit message should contain the key of the JIRA issue.
68
-
* For example, `[COMPRESS-123] Close input stream earlier`
68
+
* For example, `[COMPRESS-123] Close input stream sooner`
69
69
+ Respect the original code style:
70
70
+ Only use spaces for indentation; you can check for unnecessary whitespace with `git diff` before committing.
71
71
+ Create minimal diffs - disable _On Save_ actions like _Reformat Source Code_ or _Organize Imports_. If you feel the source code should be reformatted create a separate PR for this change first.
@@ -71,7 +71,7 @@ Alternatively, you can pull it from the central Maven repositories:
71
71
<dependency>
72
72
<groupId>org.apache.commons</groupId>
73
73
<artifactId>commons-compress</artifactId>
74
-
<version>1.27.1</version>
74
+
<version>1.28.0</version>
75
75
</dependency>
76
76
```
77
77
@@ -93,7 +93,7 @@ There are some guidelines which will make applying PRs easier for us:
93
93
+ Create minimal diffs - disable on save actions like reformat source code or organize imports. If you feel the source code should be reformatted create a separate PR for this change.
94
94
+ Provide JUnit tests for your changes and make sure your changes don't break any existing tests by running `mvn`.
95
95
+ Before you pushing a PR, run `mvn` (by itself), this runs the default goal, which contains all build checks.
96
-
+ To see the code coverage report, regardless of coverage failures, run `mvn clean site -Dcommons.jacoco.haltOnFailure=false`
96
+
+ To see the code coverage report, regardless of coverage failures, run `mvn clean site -Dcommons.jacoco.haltOnFailure=false -Pjacoco`
97
97
98
98
If you plan to contribute on a regular basis, please consider filing a [contributor license agreement](https://www.apache.org/licenses/#clas).
99
99
You can learn more about contributing via GitHub in our [contribution guidelines](CONTRIBUTING.md).
0 commit comments