Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated Contributing doc to reflect the requirement for signed commits #5352

Merged
merged 2 commits into from Sep 15, 2022

Conversation

castelblanque
Copy link
Collaborator

Signed-off-by: Rafa Castelblanque rcastelblanq@vmware.com

Description of the change

Adds the requirement for contributors to sign (with GPG) and sign off the Git commits of contributors.
After some PRs were already created by contributors, we realized that commits were not signed, which is a requirement now for Kubeapps Git repo.

Benefits

Less confusion for contributors when making contributions.

Possible drawbacks

N/A

Applicable issues

Additional information

Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
@netlify
Copy link

netlify bot commented Sep 15, 2022

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit 9c93f2f
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/6322d079f91aa6000886262a

Copy link
Contributor

@antgamdia antgamdia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@castelblanque castelblanque merged commit 141e28b into main Sep 15, 2022
@castelblanque castelblanque deleted the updated-contributing branch September 15, 2022 07:54
beni0888 pushed a commit to beni0888/kubeapps that referenced this pull request Sep 19, 2022
vmware-tanzu#5352)

Updated Contributing doc

Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>

Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants