The following is a set of guidelines for contributing to the NGINX Ingress Operator. We really appreciate that you are considering contributing!
Please open an Issue on GitHub with the label question
.
Follow our Installation Guide to get the NGINX Ingress Operator up and running.
Read the documentation and examples.
- This Operator is written in Go using the operator-framework and supports both the open source NGINX Ingress Controller and NGINX Plus Ingress Controller.
- The project follows a standard Go project layout
- The main code is found at
cmd/manager/
- The operator code is found at
pkg/
- Build files for Docker and CI are found under
build/
- We use Go Modules for managing dependencies.
- The main code is found at
To report a bug, open an issue on GitHub with the label bug
using the available bug report issue template. Please ensure the issue has not already been reported.
To suggest an enhancement, please create an issue on GitHub with the label enhancement
using the available feature issue template.
- Fork the repo, create a branch, submit a PR when your changes are tested and ready for review
- Fill in our pull request template
Note: if you’d like to implement a new feature, please consider creating a feature request issue first to start a discussion about the feature.
- Keep a clean, concise and meaningful git commit history on your branch, rebasing locally and squashing before submitting a PR
- Follow the guidelines of writing a good commit message as described here https://chris.beams.io/posts/git-commit/ and summarised in the next few points
- In the subject line, use the present tense ("Add feature" not "Added feature")
- In the subject line, use the imperative mood ("Move cursor to..." not "Moves cursor to...")
- Limit the subject line to 72 characters or less
- Reference issues and pull requests liberally after the subject line
- Add more detailed description in the body of the git message (
git commit -a
to give you more space and time in your text editor to write a good message instead ofgit commit -am
)
- Run
gofmt
over your code to automatically resolve a lot of style issues. Most editors support this running automatically when saving a code file. - Follow this guide on some good practice and idioms for Go - https://github.com/golang/go/wiki/CodeReviewComments
- To check for extra issues, install golangci-lint and run
make lint
orgolangci-lint run