Skip to content

Latest commit

 

History

History
83 lines (57 loc) · 3.92 KB

CONTRIBUTING.md

File metadata and controls

83 lines (57 loc) · 3.92 KB
Raw

Contribute to scaleway-cloud-controller-manager

scaleway-cloud-controller-manager is Apache 2.0 licensed and accepts contributions via GitHub. This document will cover how to contribute to the project and report issues.

Topics

Reporting security issues

At Scaleway we take security seriously. If you have any issue regarding security, please notify us by sending an email to [email protected].

Please DO NOT create a GitHub issue.

We will follow up with you promptly with more information and a plan for remediation. We currently do not offer a paid security bounty program, but we would love to send some Scaleway swag your way along with our deepest gratitude for your assistance in making Scaleway a more secure Cloud ecosystem.

Reporting issues

A great way to contribute to the project is to send a detailed report when you encounter a bug. We always appreciate a well-written, thorough bug report, and will thank you for it! Before opening a new issue, we appreciate you reviewing open issues to see if there are any similar requests. If there is a match, thumbs up the issue with a 👍 and leave a comment if you have additional information.

When reporting an issue, include the following:

  • The version of scaleway-cloud-controller-manager you are using (v0.1.3, v0.2.4, master, ...)
  • Kubernetes apiserver version, flags and environments used.
  • Kubelet version, flags and environments used.
  • Go version
  • GOOS
  • GOARCH

Suggesting a feature

When requesting a feature, some of the questions we want to answer are:

  • What value does this feature bring to end users ?
  • How urgent is the need (nice to have feature or need to have) ?
  • Does this align with the goals of scaleway-cloud-controller-manager ?

Submit code

To submit code:

  • Create a fork of the project
  • Create a topic branch from where you want to base your work (usually master)
  • Add tests to cover contributed code
  • Push your commit(s) to your topic branch on your fork
  • Open a pull request against scaleway-cloud-controller-manager master branch that follows PR guidelines

The maintainers of scaleway-cloud-controller-manager use a "Let's Get This Merged" (LGTM) message in the pull request to note that the commits are ready to merge. After one or more maintainer states LGTM, we will merge. If you have questions or comments on your code, feel free to correct these in your branch through new commits.

Pull Request Guidelines

The goal of the following guidelines is to have Pull Requests (PRs) that are fairly easy to review and comprehend, and code that is easy to maintain in the future.

  • Pull Request title should be clear on what is being fixed or added to the code base. If you are addressing an open issue, please start the title with "fix: #XXX" or "feature: #XXX"
  • Keep it readable for human reviewers and prefer a subset of functionality (code) with tests and documentation over delivering them separately
  • Don't forget commenting code to help reviewers understand your code
  • Notify Work In Progress PRs by prefixing the title with [WIP]
  • Please, keep us updated. We will try our best to merge your PR, but please notice that PRs may be closed after 30 days of inactivity.

Your pull request should be rebased against the current master branch. Please do not merge the current master branch in with your topic branch, nor use the Update Branch button provided by GitHub on the pull request page.

Keep in mind only the Pull Request Title will be used as commit message as we stash all commits on merge.

Community guidelines

Thank you for reading through all of this, if you have any question feel free to reach us!