-
Notifications
You must be signed in to change notification settings - Fork 186
Working with the Code
See the CONTRIBUTING
filec file in the repository for information on how to contribute issues and code changes to the project.
Spring Vault requires SSL certificates and a running Vault instance listening on localhost:8200
.
Certificates and the Vault setup are scripted, the scripts are located in src/test/bash
.
The following scripts need to be run prior to building the project for the tests to pass.
$ ./src/test/bash/install_vault.sh $ ./src/test/bash/create_certificates.sh $ ./src/test/bash/env.sh $ ./src/test/bash/local_run_vault.sh
Alternatively, you can run
$ ./src/test/bash/start.sh
Changes to the documentation should be made to the adocs found under src/main/asciidoc/
To build the source you will need to install JDK 1.6.
Spring Vault uses Maven for most build-related activities, and you should be able to get off the ground quite quickly by cloning the project you are interested in and typing
$ ./mvnw install
Note
|
You can also install Maven (>=3.3.3) yourself and run the mvn command in place of ./mvnw in the examples below.
If you do that you also might need to add -P spring if your local Maven settings do not contain repository declarations for spring pre-release artifacts.
|
Note
|
Be aware that you might need to increase the amount of memory available to Maven by setting a MAVEN_OPTS environment variable with a value like -Xmx512m -XX:MaxPermSize=128m .
We try to cover this in the .mvn configuration, so if you find you have to do it to make a build succeed, please raise a ticket to get the settings added to source control.
|
For hints on how to build the project look in .travis.yml
if there is one.
There should be a "script" and maybe "install" command.
Also look at the "services" section to see if any services need to be running locally (e.g. mongo or rabbit).
Ignore the git-related bits that you might find in "before_install" since they’re related to setting git credentials and you already have those.
Note
|
If all else fails, build with the command from .travis.yml (usually
./mvnw install ).
|
The module has a "distribute" profile, and if you switch that on it will try to build asciidoc sources from
src/main/asciidoc
.
If you don’t have an IDE preference we would recommend that you use Spring Tools Suite or Eclipse when working with the code. We use the m2eclipe eclipse plugin for maven support. Other IDEs and tools should also work without issue as long as they use Maven 3.3.3 or better.
We recommend the m2eclipe eclipse plugin when working with eclipse. If you don’t already have m2eclipse installed it is available from the "eclipse marketplace".
Note
|
Older versions of m2e do not support Maven 3.3, so once the projects are imported into Eclipse you will also need to tell m2eclipse to use the right profile for the projects.If you see many different errors related to the POMs in the projects, check that you have an up to date installation.
If you can’t upgrade m2e, add the "spring" profile to your settings.xml .
Alternatively you can copy the repository settings from the "spring" profile of the parent pom into your settings.xml .
|
Spring Vault is released under the non-restrictive Apache 2.0 license, and follows a very standard Github development process, using Github tracker for issues and merging pull requests into main
.
If you want to contribute even something trivial please do not hesitate, but follow the guidelines below.
Before we accept a non-trivial patch or pull request we will need you to sign the Contributor License Agreement. 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. Active contributors might be asked to join the core team, and given the ability to merge pull requests.
This project adheres to the Contributor Covenant code of conduct. By participating, you are expected to uphold this code. Please report unacceptable behavior to [email protected].
None of these is essential for a pull request, but they will all help. They can also be added after the original pull request but before a merge.
-
Spring Vault uses the Spring JavaFormat conventions. Formatting is applied when running the build through
$ ./mvnw compile
IDE plugins are available from https://github.com/spring-io/spring-javaformat. -
Make sure all new
.java
files to 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. -
Add the ASF license header comment to all new
.java
files (copy from existing files in the project) -
Add yourself as an
@author
to the .java files that you modify substantially (more than cosmetic changes). -
Please include unit tests.
-
If no-one else is using your branch, please rebase it against the current
main
(or other target branch in the main project). -
When writing a commit message please follow these conventions, if you are fixing an existing issue please add
Fixes gh-XXXX
at the end of the commit message (where XXXX is the issue number).