Thank you for your interest in contributing to our project. Whether it’s a bug report, new feature, correction, or additional documentation, we greatly value feedback and contributions from our community.
Please read through this document before submitting any issues or pull requests to ensure we have all the necessary information to effectively respond to your bug report or contribution.
Before you get started, we ask that you:
- Check out the code of conduct.
- Sign the lakeFS CLA when making your first pull request (individual / corporate)
- Submit any security issues directly to firstname.lastname@example.org.
- Contributions should have an associated GitHub issue.
- Before making major contributions, please reach out to us on the #dev channel on Slack. We will make sure no one else is working on the same feature.
This section was tested on macOS and Linux (Fedora 32, Ubuntu 20.04) - Your mileage may vary
Our Go release workflow holds the Go and Node.js versions we currently use under go-version and node-version compatibly. The Java workflows use Maven 3.8.1 (but any recent version of Maven should work).
- Install the required dependencies for your OS:
go get github.com/rakyll/statik
(go env GOPATH)/binis in your
$PATH(or at least, that the
- Clone the repository from https://github.com/treeverse/lakeFS (gives you read-only access to the repository. To contribute, see the next section).
Build the project:
Make sure tests are passing:
- Review this document in full
- Make sure there’s an open issue on GitHub that this pull request addresses, and that it isn’t labeled
- Fork the lakeFS repository
- If you’re adding new functionality, create a new branch named
- If you’re fixing a bug, create a new branch named
fix/<DESCRIPTIVE NAME>-<ISSUE NUMBER>
Once you’ve made the necessary changes to the code, make sure tests pass:
Check linting rules are passing:
lakeFS uses go fmt as a style guide for Go code.
Open a GitHub pull request with your change. The PR description should include a brief explanation of your change. You should also mention the related GitHub issue. If the issue should be automatically closed after the merge, please link it to the PR.
After submitting your pull request, GitHub Actions will automatically run tests on your changes and make sure that your updated code builds and runs on Go 1.16.2.
Check back shortly after submitting your pull request to make sure that your code passes these checks. If any of the checks come back with a red X, then do your best to address the errors.
A developer from our team will review your pull request, and may request some changes to it. After the request is approved, it will be merged to our main branch.
Documentation of features and changes in behaviour should be included in the pull-request. You can create separate pull requests for documentation changes only. Documentation site customizations should be performed in accordance with the Just The Docs Customization guide, which is applied during the site creation process.
Any user-facing change should be included in the
CHANGELOG.md file, under the “Unreleased” section at the top.
When we publish a new version of lakeFS, we will move this to the relevant version section.
The changelog entry should contain a concise description of the feature or fix, along with the GitHub issue number.