Skip to content

Commit 20e4db0

Browse files
royadityak94oscerd
authored andcommitted
Proofread CONTRIBUTING file with appropriate edits
Made necessary typo, grammatical, punctuation fixes at several places in the CONTRIBUTING.md file.
1 parent d39c45c commit 20e4db0

File tree

1 file changed

+36
-35
lines changed

1 file changed

+36
-35
lines changed

CONTRIBUTING.md

Lines changed: 36 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
## Contributing to Apache Camel
22

33
There are many ways you can help make Camel a better piece of software - please dive in and help!
4-
- Try surf the documentation - if somethings confusing or not clear, let us know.
4+
- Try surfing the documentation - if something confuses you, bring it to our attention.
55
- Download the code & try it out and see what you think.
6-
- Browse the source code. Got an itch to scratch, want to tune some operation or add some feature?
7-
- Want to do some hacking on Camel? Try surfing the our [issue tracker](https://issues.apache.org/jira/browse/CAMEL) for open issues or features that need to be implemented, take ownership of an issue and try fix it.
6+
- Browse the source code. Got an itch to scratch, want to tune some operation, or add some feature?
7+
- Want to do some hacking on Camel? Try surfing our [issue tracker](https://issues.apache.org/jira/browse/CAMEL) for open issues or features that need to be implemented, take active ownership of a particular issue, and try to fix it.
88
- If you are a new Camel rider and would like to help us, you can also find [some easy to resolve issues.](https://issues.apache.org/jira/issues/?filter=12348074)
9-
- Leave a comment on the issue to let us know you are working on it and add yourself as a watcher to get informed about all modifications.
9+
- Leave a comment on the issue to let us know you are working on it, and add yourself as a watcher to get informed about all modifications.
1010

1111

1212
## Table of Contents
@@ -35,30 +35,30 @@ There are various ways of communicating with the Camel community.
3535

3636
## Improving the documentation
3737

38-
Documentation is massively important to help users make the most of Apache Camel and its probably the area that needs the most help!
39-
So if you are interested in helping the documentation effort; whether its just to fix a page here or there, correct a link or even write a tutorial or improve what documentation is already there please do dive in and help!
38+
Documentation is massively critical for users intending to make the most of Apache Camel, and it's probably the area that needs the most help!
39+
So if you are interested in helping with the documentation efforts, whether it's just to fix a page here or there, correct a link, or even write a tutorial or improve existing documentation, please dive in and help!
4040
We moved the documentation into the code (AsciiDoc). We are not using the wiki system anymore.
4141

4242
To edit the documentation:
4343
- It's easy as opening a Pull Request
44-
- You'll find on each component under src/main/docs an .adoc file
45-
- This file contains a static part and a dynamically generated part: the former can be edited directly in the .adoc file, while the latter needs your intervention on the javadoc
46-
- Once you modify the javadoc, you'll need to rebuild the component and the .adoc will be automatically updated
44+
- You'll find on each component under src/main/docs a .adoc file
45+
- This file contains a static part and a dynamically generated part: the former can be edited directly in the .adoc file, while the latter needs your intervention on the Javadoc
46+
- Once you modify the Javadoc, you'll need to rebuild the component. And, the .adoc file will get automatically updated.
4747
- Create a commit and raise a Pull Request
48-
- If you want to add more documentation check for .adoc in codebase
48+
- If you want to add more documentation, find the respective .adoc in the codebase
4949

50-
For more information see [How does the website work](https://camel.apache.org/manual/latest/faq/how-does-the-website-work.html) or [How do I edit the website for more details](https://camel.apache.org/manual/latest/faq/how-do-i-edit-the-website.html).
50+
For more information, see [How does the website work](https://camel.apache.org/manual/latest/faq/how-does-the-website-work.html) or [How do I edit the website for more details](https://camel.apache.org/manual/latest/faq/how-do-i-edit-the-website.html).
5151

5252
## If you find a bug or problem
5353

5454
Please raise a new issue in our [issue tracker](https://issues.apache.org/jira/browse/CAMEL)
55-
If you can create a JUnit test case then your issue is more likely to be resolved quicker.
56-
e.g. take a look at some of the existing [unit tests cases](https://github.com/apache/camel/tree/master/core/camel-core/src/test/java/org/apache/camel)
55+
If you can create a JUnit test case, then your issue is more likely to be resolved quicker.
56+
e.g., take a look at some of the existing [unit tests cases](https://github.com/apache/camel/tree/master/core/camel-core/src/test/java/org/apache/camel)
5757

5858

5959
## Working on the code
6060

61-
We recommend to work on the code from [github](https://github.com/apache/camel/).
61+
We recommend working on the code from [github](https://github.com/apache/camel/).
6262

6363
git clone https://github.com/apache/camel.git
6464
cd camel
@@ -67,29 +67,30 @@ Build the project (fast build).
6767

6868
mvn clean install -Pfastinstall
6969

70-
If you intend to work on the code and provide patches and other work you want to submit to the Apache Camel project, then you can fork the project on github and work on your own fork. The custom work you do should be done on branches you create, which can then be committed and pushed upstream, and then submitted to Apache Camel as PRs (pull requests). You can find many resources online how to work on github projects and how to submit work to these projects.
70+
If you intend to work on the code and provide patches and other work you want to submit to the Apache Camel project, then you can fork the project on GitHub and work on your fork. Your custom-work needs to be on your self-created branches, which can then be committed and pushed upstream, and then submitted to Apache Camel as PRs (pull requests). You can find many resources online on how to work on GitHub projects and how to submit work to these projects.
7171

72-
Please avoid unnecessary changes, like reordering methods and fields. This will make your PR easier to review.
72+
Please avoid unnecessary changes, like reordering methods and fields, which will make your PR easier to review.
7373

7474

7575
## Running checkstyle
7676

77-
Apache Camel source code is using a coding style/format which can be checked whether is complying using the checkstyle plugin.
78-
To enable source style checking with checkstyle, build Camel with the -Psourcecheck parameter
77+
Apache Camel source code is using a coding style/format that can
78+
be verified for its compliance using the checkstyle plugin.
79+
To enable source style checking with checkstyle, build Camel with the -Psourcecheck parameter:
7980

8081
mvn clean install -Psourcecheck
8182

82-
Please remember to run this check on your code changes before submitting a patch or github PR. You do not need to run this against the entire project, but for example in the modules you work on. Lets say you do some code changes in the camel-ftp component, then you can run the check from within this directory:
83+
Please remember to run this check on your code changes before submitting a patch or Github PR. You do not need to run this against the entire project, but only in your modules. Let's say you do some code changes in the camel-ftp component, following which you can run the check from within this directory:
8384

8485
cd camel-ftp
8586
mvn clean install -Psourcecheck
8687

8788
## Verify Karaf features
8889

89-
Camel-Karaf lives now in his own repository, so to verify a Karaf feature you'll need to fork the following [repository](https://github.com/apache/camel-karaf).
90+
Camel-Karaf now lives in its self repository, so to verify a Karaf feature, you'll need to fork the following [repository](https://github.com/apache/camel-karaf).
9091

91-
To check a new Karaf feature or an existing one you should run a verification on the features.xml file. You'll need to follow these steps:
92-
First thing to be done is running a full build of Camel. Then
92+
To check a new Karaf feature or an existing one, you should run a verification on the features.xml file. You'll need to follow these steps:
93+
The first thing to be done is running a full build of Camel. Then
9394

9495
cd platform/karaf/features/
9596
mvn clean install
@@ -98,7 +99,7 @@ If you modified a component/dataformat or updated a dependency in the main camel
9899

99100
## Apache Camel committers should work on the ASF git repo
100101

101-
If you are an Apache Camel committer then clone the ASF git repo at
102+
If you are an Apache Camel committer, then clone the ASF git repo at
102103

103104
git clone https://gitbox.apache.org/repos/asf/camel.git
104105
cd camel
@@ -112,31 +113,31 @@ Build the project (without testing).
112113

113114
mvn clean install -Dtest=false
114115

115-
PS: You might need to build multiple times (if you get a build error) because sometimes maven fails to download all the files.
116+
PS: You might need to build multiple times (if you get a build error) because sometimes the maven fails to download all the files.
116117
Then import the projects into your workspace.
117118

118119

119120
## Creating patches
120121

121-
We recommend you create patches as github PRs which is much easier for us to accept and work with. You do this as any other github project, where you can fork the project, and create a branch where you work on the code, and then commit and push that code to your fork. Then navigate to the Apache Camel github webpage, and you will see that github in the top of the page has a wizard to send your recent work as a PR (pull request).
122+
We recommend you create patches as GitHub PRs as it eases our reviewing process enables faster review completion. You do this as any other GitHub project, where you can fork the project, and create a branch where you work on the code, and then commit and push that code to your fork. Then navigate to the Apache Camel GitHub webpage, and you will see that GitHub at the top of the page has a wizard to send your recent work as a PR (pull request).
122123

123124

124125
## Pull request at Github
125126

126-
There is also a Git repository at Github which you could fork. Then you submit patches as any other github project - eg work on a new feature branch and send a pull request. One of the committers then needs to accept your pull request to bring the code to the ASF codebase.
127+
There is also a Git repository at GitHub which you could fork. Then you submit patches as any other GitHub project - e.g., work on a new feature branch and send a pull request. One of the committers then needs to accept your pull request to bring the code to the ASF codebase.
127128

128-
When providing code patches then please include the Camel JIRA ticket number in the commit messages.
129+
When providing code patches, please include the Camel JIRA ticket number in the commit messages.
129130
We favor using the syntax:
130131

131132
CAMEL-9999: Some message goes here
132133

133134
## Manual patch files
134-
We gladly accept patches if you can find ways to improve, tune or fix Camel in some way.
135+
We gladly accept patches if you can find ways to improve, tune, or fix Camel in some way.
135136

136-
We recommend using github PRs instead of manual patch files. Especially for bigger patches.
137+
We recommend using GitHub PRs instead of manual patch files. Especially for larger patches.
137138

138-
Most IDEs can create nice patches now very easily. e.g. in Eclipse just right click on a file/directory and select Team -> Create Patch. Then just save the patch as a file and attach it to the corresponding issue on our [JIRA issue tracker](https://issues.apache.org/jira/browse/CAMEL).
139-
If you're a command line person try the following to create the patch
139+
Most IDEs can create nice patches now very easily. e.g., on Eclipse, right-click on a file/directory, and select Team -> Create Patch. Then, save the patch as a file and attach it to the corresponding issue on our [JIRA issue tracker](https://issues.apache.org/jira/browse/CAMEL).
140+
If you prefer working on the command-line, try the following to create the patch:
140141

141142
diff -u Main.java.orig Main.java >> patchfile.txt
142143

@@ -149,18 +150,18 @@ or
149150

150151
The easiest way to submit a patch is to
151152
- [create a new JIRA issue](https://issues.apache.org/jira/browse/CAMEL) (you will need to register),
152-
- attach the patch or tarball as an attachment (if you create a patch file, but we recommend using github PRs)
153+
- attach the patch or tarball as an attachment (if you create a patch file, but we recommend using GitHub PRs)
153154
- **tick the Patch Attached** button on the issue
154-
We prefer patches has unit tests as well and that these unit tests have proper assertions as well, so remember to replace your system.out or logging with an assertion instead!
155+
We prefer patches to have unit tests as well and that these unit tests have proper assertions as well, so remember to replace your system.out or logging with appropriate assertions.
155156

156157

157158
## Using the issue tracker
158159

159-
Before you can raise an issue in the [issue tracker](https://issues.apache.org/jira/browse/CAMEL) you need to register with it. This is quick & painless.
160+
Before you can raise an issue in the [issue tracker](https://issues.apache.org/jira/browse/CAMEL), you need to register with it: This is quick & painless.
160161

161162

162163
## Becoming a committer
163164

164165
Once you've got involved as above, we may well invite you to be a committer. See [How do I become a committer](https://camel.apache.org/manual/latest/faq/how-do-i-become-a-committer.html) for more details.
165166

166-
The first step is contributing to the project; if you want to take that a step forward and become a fellow committer on the project then see the [Committer Guide](https://camel.apache.org/manual/latest/faq/how-do-i-become-a-committer.html)
167+
The first step is contributing to the project; if you want to take that a step forward and become a fellow committer on the project, please check out our [Committer Guide](https://camel.apache.org/manual/latest/faq/how-do-i-become-a-committer.html).

0 commit comments

Comments
 (0)