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-init
The 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/go
A 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
--signoff
or-s
(lowercase) flag when making each commit. For example:git commit --message "<commit message>" --signoff
If you ever make a commit and forget to use the
--signoff
flag, 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
Author
andSigned-off-by
lines 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.