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

Added new templates. #454

Merged
merged 1 commit into from
Jul 2, 2018

Conversation

jhamilton1
Copy link
Contributor

What this PR does / why we need it:
The PR adds standardized bug and feature-enhancement templates. Starting with sonobuoy we will apply these to all repos.

Which issue(s) this PR fixes

  • Fixes #
    N/A

Special notes for your reviewer:
I will start a document to track which repos have been updated with these templates.

Release note:
None

release-note
None

Copy link
Contributor

@chuckha chuckha left a comment

Choose a reason for hiding this comment

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

I think this looks good, thanks for working on this.

I left one optional comment, feel free to ignore it, it might be too nit-picky.


- Sonobuoy version:
- Kubernetes version: (use `kubectl version`):
- Kubernetes installer & version:
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe this could be: Kubernetes installer & installer version (e.g. kops, kubeadm, kubespray)

@chuckha
Copy link
Contributor

chuckha commented Jul 2, 2018

Oh, @jhamilton1 you will have to sign off your commit with git. Also please squash down to 1 commit to get a nice commit message otherwise we'll squash this side and all the commit messages will be concatenated and end up in one big commit message.

Thank you for working on this!

Copy link
Contributor

@chuckha chuckha left a comment

Choose a reason for hiding this comment

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

Realized there were a few things I forgot to check before approving -- please update those and then it lgtm

@jhamilton1
Copy link
Contributor Author

Okay, I will get that straightened out.

@jhamilton1
Copy link
Contributor Author

Okay, @chuckha I squashed down to 1 commit and signed that off. Looks like I am still failing that check though.

@timothysc
Copy link
Contributor

@jhamilton1 - nope now you have 5 commits.

Signed-off-by: Jesse <jesse.hamilton@heptio.com>
@jhamilton1 jhamilton1 force-pushed the jhamilton1-new-templates branch from 9a10b45 to 61abe9c Compare July 2, 2018 17:10
Copy link
Contributor

@timothysc timothysc left a comment

Choose a reason for hiding this comment

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

/lgtm

@timothysc timothysc merged commit 2d3b398 into vmware-tanzu:master Jul 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants