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

docs(helm): Mention commit signing in the Developer Guide #5018

Merged
merged 1 commit into from
Dec 5, 2018

Conversation

henrynash
Copy link

@henrynash henrynash commented Dec 5, 2018

This change adds a mention to the requirement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes #5016

Signed-off-by: Henry Nash henry.nash@uk.ibm.com

@helm-bot helm-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 5, 2018
@bacongobbler bacongobbler changed the title dosc(helm): Mention commit signing in the Developer Guide docs(helm): Mention commit signing in the Developer Guide Dec 5, 2018
@bacongobbler
Copy link
Member

there's a slight typo in your commit message if you wanna fix that. "dosc" should be "docs". Otherwise LGTM

@henrynash
Copy link
Author

@bacongobbler well spotted, fix on its way

This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
@helm-bot helm-bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 5, 2018
@bacongobbler bacongobbler merged commit c80f7dd into helm:master Dec 5, 2018
splisson pushed a commit to splisson/helm that referenced this pull request Dec 6, 2018
This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
Signed-off-by: Sebastien Plisson <sebastien.plisson@gmail.com>
splisson pushed a commit to splisson/helm that referenced this pull request Dec 6, 2018
This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
Signed-off-by: Sebastien Plisson <sebastien.plisson@gmail.com>
splisson pushed a commit to splisson/helm that referenced this pull request Dec 6, 2018
This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
splisson pushed a commit to splisson/helm that referenced this pull request Dec 6, 2018
This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
consolewitch pushed a commit to consolewitch/helm that referenced this pull request Jan 7, 2019
This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
eraac pushed a commit to eraac/helm that referenced this pull request Mar 21, 2019
This change adds a mention to the reqruiement for commit signing
to the Git Convention outline in the Developer Guide, while
leaving the existing detail on signing in the Contributing doc.

Closes helm#5016

Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>
Signed-off-by: Kevin Labesse <kevin@labesse.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants