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
+11-8Lines changed: 11 additions & 8 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -6,7 +6,7 @@
6
6
(the "License"); you may not use this file except in compliance with
7
7
the License. You may obtain a copy of the License at
8
8
9
-
http://www.apache.org/licenses/LICENSE-2.0
9
+
https://www.apache.org/licenses/LICENSE-2.0
10
10
11
11
Unless required by applicable law or agreed to in writing, software
12
12
distributed under the License is distributed on an "AS IS" BASIS,
@@ -41,9 +41,8 @@
41
41
Contributing to Apache Commons Imaging
42
42
======================
43
43
44
-
You have found a bug or you have an idea for a cool new feature? Contributing code is a great way to give something back to
45
-
the open source community. Before you dig right into the code there are a few guidelines that we need contributors to
46
-
follow so that we can have a chance of keeping on top of things.
44
+
Have you found a bug or have an idea for a cool new feature? Contributing code is a great way to give something back to the open-source community.
45
+
Before you dig right into the code, we need contributors to follow a few guidelines to have a chance of keeping on top of things.
47
46
48
47
Getting Started
49
48
---------------
@@ -62,16 +61,20 @@ Making Changes
62
61
63
62
+ Create a _topic branch_ for your isolated work.
64
63
* Usually you should base your branch from the `master` branch.
65
-
* A good topic branch name can be the JIRA bug id plus a keyword, for example,`IMAGING-123-InputStream`.
64
+
* A good topic branch name can be the JIRA bug ID plus a keyword, e.g.`IMAGING-123-InputStream`.
66
65
* If you have submitted multiple JIRA issues, try to maintain separate branches and pull requests.
67
66
+ Make commits of logical units.
68
67
* Make sure your commit messages are meaningful and in the proper format. Your commit message should contain the key of the JIRA issue.
69
-
* For example, `[IMAGING-123] Close input stream earlier`
68
+
* For example, `[IMAGING-123] Close input stream sooner`
70
69
+ Respect the original code style:
71
70
+ Only use spaces for indentation; you can check for unnecessary whitespace with `git diff` before committing.
72
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.
73
-
+ Make sure you have added the necessary tests for your changes, typically in `src/test/java`.
74
-
+ Run all the tests with `mvn clean verify` to assure nothing else was accidentally broken.
72
+
+ Write unit tests that match behavioral changes, where the tests fail if the changes to the runtime are not applied. This may not always be possible but is a best-practice.
73
+
Unit tests are typically in the `src/test/java` directory.
74
+
+ Run a successful build using the default [Maven](https://maven.apache.org/) goal with `mvn`; that's `mvn` on the command line by itself.
75
+
+ Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
76
+
+ Each commit in the pull request should have a meaningful subject line and body. Note that commits might be squashed by a maintainer on merge.
You can download source and binaries from our [download page](https://commons.apache.org/proper/commons-imaging/download_imaging.cgi).
64
64
65
-
Alternatively, you can pull it from the central Maven repositories:
65
+
Alternatively, you can pull it from the central Maven repositories:
66
66
67
67
```xml
68
68
<dependency>
69
69
<groupId>org.apache.commons</groupId>
70
70
<artifactId>commons-imaging</artifactId>
71
-
<version>1.0.0-alpha5</version>
71
+
<version>1.0.0-alpha6</version>
72
72
</dependency>
73
73
```
74
74
75
75
Building
76
76
--------
77
77
78
-
Building requires a Java JDK and [Apache Maven](https://maven.apache.org/).
78
+
Building requires a Java JDK and [Apache Maven](https://maven.apache.org/).
79
79
The required Java version is found in the `pom.xml` as the `maven.compiler.source` property.
80
80
81
81
From a command shell, run `mvn` without arguments to invoke the default Maven goal to run all tests and checks.
@@ -88,7 +88,9 @@ There are some guidelines which will make applying PRs easier for us:
88
88
+ No tabs! Please use spaces for indentation.
89
89
+ Respect the existing code style for each file.
90
90
+ 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.
91
-
+ Provide JUnit tests for your changes and make sure your changes don't break any existing tests by running ```mvn```.
91
+
+ Provide JUnit tests for your changes and make sure your changes don't break any existing tests by running `mvn`.
92
+
+ Before you pushing a PR, run `mvn` (by itself), this runs the default goal, which contains all build checks.
93
+
+ To see the code coverage report, regardless of coverage failures, run `mvn clean site -Dcommons.jacoco.haltOnFailure=false -Pjacoco`
92
94
93
95
If you plan to contribute on a regular basis, please consider filing a [contributor license agreement](https://www.apache.org/licenses/#clas).
94
96
You can learn more about contributing via GitHub in our [contribution guidelines](CONTRIBUTING.md).
If you would like to report a bug, or raise an enhancement request with
66
68
Apache Commons Imaging please do the following:
69
+
</p>
67
70
<ol>
68
71
<li><ahref="https://issues.apache.org/jira/secure/IssueNavigator.jspa?reset=true&pid=12313421&sorter/field=issuekey&sorter/order=DESC&status=1&status=3&status=4">Search existing open bugs</a>.
69
72
If you find your issue listed then please add a comment with your details.</li>
@@ -73,16 +76,15 @@ limitations under the License.
73
76
<li>Submit either a <ahref="https://issues.apache.org/jira/secure/CreateIssueDetails!init.jspa?pid=12313421&issuetype=1&priority=4&assignee=-1">bug report</a>
74
77
or <ahref="https://issues.apache.org/jira/secure/CreateIssueDetails!init.jspa?pid=12313421&issuetype=4&priority=4&assignee=-1">enhancement request</a>.</li>
75
78
</ol>
76
-
</p>
77
79
78
80
<p>
79
81
Please also remember these points:
82
+
</p>
80
83
<ul>
81
84
<li>the more information you provide, the better we can help you</li>
82
85
<li>test cases are vital, particularly for any proposed enhancements</li>
83
86
<li>the developers of Apache Commons Imaging are all unpaid volunteers</li>
84
87
</ul>
85
-
</p>
86
88
87
89
<p>
88
90
For more information on creating patches see the
@@ -91,12 +93,12 @@ limitations under the License.
91
93
92
94
<p>
93
95
You may also find these links useful:
96
+
</p>
94
97
<ul>
95
98
<li><ahref="https://issues.apache.org/jira/secure/IssueNavigator.jspa?reset=true&pid=12313421&sorter/field=issuekey&sorter/order=DESC&status=1&status=3&status=4">All Open Apache Commons Imaging bugs</a></li>
0 commit comments