There are a few things to consider before contributing to flagd.
Firstly, there's a code of conduct. TLDR: be respectful.
Any contributions are expected to include unit tests.
These can be validated with make test or the automated github workflow will run them on PR creation.
This project uses a go workspace, to setup the project run
make workspace-initThe go version in the go.work is the currently supported version of go.
The project uses remote buf packages, changing the remote generation source will require a one-time registry configuration:
export GOPRIVATE=buf.build/gen/goA DCO (Developer Certificate of Origin) sign-off is a line placed at the end of a commit message containing a contributor's "signature." In adding this, the contributor certifies that they have the right to contribute the material in question.
Here are the steps to sign your work:
-
Verify the contribution in your commit complies with the terms of the DCO.
-
Add a line like the following to your commit message:
Signed-off-by: Joe Smith <joe.smith@example.com>
You MUST use your legal name -- handles or other pseudonyms are not permitted.
While you could manually add DCO sign-off to every commit, there is an easier way:
-
Configure your git client appropriately. This is one-time setup.
git config user.name <legal name> git config user.email <email address you use for GitHub>
If you work on multiple projects that require a DCO sign-off, you can configure your git client to use these settings globally instead of only for Brigade:
git config --global user.name <legal name> git config --global user.email <email address you use for GitHub>
-
Use the
--signoffor-s(lowercase) flag when making each commit. For example:git commit --message "<commit message>" --signoffIf you ever make a commit and forget to use the
--signoffflag, you can amend your commit with this information before pushing:git commit --amend --signoff
-
You can verify the above worked as expected using
git log. Your latest commit should look similar to this one:Author: Joe Smith <joe.smith@example.com> Date: Thu Feb 2 11:41:15 2018 -0800 Update README Signed-off-by: Joe Smith <joe.smith@example.com>
Notice the
AuthorandSigned-off-bylines match. If they do not, the PR will be rejected by the automated DCO check.
-
When raising PRs, please format according to conventional commit standards
For example: docs: some PR title here...
Thanks! Issues and pull requests following these guidelines are welcome.
PRs are expected to conform to markdown lint rules.
Therefore, run make markdownlint-fix to auto-fix most issues.
Then commit the results.
For those issues that cannot be auto-fixed, run make markdownlint
then manually fix whatever it warns about.