Skip to content

Commit 452e456

Browse files
authored
Create CONTRIBUTING.adoc
1 parent 2e4aec6 commit 452e456

File tree

1 file changed

+58
-0
lines changed

1 file changed

+58
-0
lines changed

CONTRIBUTING.adoc

Lines changed: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
1+
= Contributing to Spring Vault
2+
3+
Spring Vault is released under the Apache 2.0 license. If you would like to contribute something or want to hack on the code this document should help you get started.
4+
5+
6+
7+
== Code of Conduct
8+
This project adheres to the Contributor Covenant link:CODE_OF_CONDUCT.adoc[code of conduct].
9+
By participating, you are expected to uphold this code. Please report unacceptable behavior to [email protected].
10+
11+
12+
13+
== Using GitHub Issues
14+
We use GitHub issues to track bugs and enhancements.
15+
If you have a general usage question please ask on https://stackoverflow.com[Stack Overflow].
16+
The Spring Vault team and the broader community monitor the https://stackoverflow.com/tags/spring-vault[`spring-vault`] tag.
17+
18+
If you are reporting a bug, please help to speed up problem diagnosis by providing as much information as possible.
19+
Ideally, that would include a small sample project that reproduces the problem.
20+
21+
22+
23+
== Reporting Security Vulnerabilities
24+
If you think you have found a security vulnerability in Spring Vault please *DO NOT* disclose it publicly until we've had a chance to fix it.
25+
Please don't report security vulnerabilities using GitHub issues, instead head over to https://spring.io/security-policy and learn how to disclose them responsibly.
26+
27+
28+
29+
== Sign the Contributor License Agreement
30+
Before we accept a non-trivial patch or pull request we will need you to https://cla.pivotal.io/sign/spring[sign the Contributor License Agreement].
31+
Signing the contributor's agreement does not grant anyone commit rights to the main repository, but it does mean that we can accept your contributions, and you will get an author credit if we do.
32+
Active contributors might be asked to join the core team, and given the ability to merge pull requests.
33+
34+
35+
36+
== Code Conventions and Housekeeping
37+
None of these is essential for a pull request, but they will all help. They can also be
38+
added after the original pull request but before a merge.
39+
40+
* We use the https://github.com/spring-io/spring-javaformat/[Spring JavaFormat] project to apply code formatting conventions.
41+
If you use Eclipse and you follow the '`Importing into eclipse`' instructions below you should get project specific formatting automatically.
42+
You can also install the https://github.com/spring-io/spring-javaformat/#intellij-idea[Spring JavaFormat IntelliJ Plugin] or format the code from the Gradle build by running `./gradlew format`.
43+
Note that if you have format violations in `buildSrc`, you can fix them by running `./gradlew -p buildSrc format` from the project root directory.
44+
* The build includes Checkstyle rules for many of our code conventions. Run `./gradlew checkstyleMain checkstyleTest` if you want to check your changes are compliant.
45+
* Make sure all new `.java` files have a Javadoc class comment with at least an `@author` tag identifying you, and preferably at least a paragraph on what the class is for.
46+
* Add the ASF license header comment to all new `.java` files (copy from existing files in the project).
47+
* Add yourself as an `@author` to the `.java` files that you modify substantially (more than cosmetic changes).
48+
* Add some Javadocs.
49+
* A few unit tests would help a lot as well -- someone has to do it.
50+
* Verification tasks, including tests and Checkstyle, can be executed by running `./gradlew check` from the project root.
51+
Note that `SPRING_PROFILES_ACTIVE` environment variable might affect the result of tests, so in that case, you can prevent it by running `unset SPRING_PROFILES_ACTIVE` before running the task.
52+
* If no-one else is using your branch, please rebase it against the current main branch (or other target branch in the project).
53+
* When writing a commit message please follow https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html[these conventions].
54+
55+
56+
57+
== Working with the Code
58+
For information on editing, building, and testing the code, see the https://github.com/spring-projects/spring-vault/wiki/Working-with-the-Code[Working with the Code] page on the project wiki.

0 commit comments

Comments
 (0)