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

If you copy pasta this it breaks. #28

Merged
merged 1 commit into from
Jan 28, 2021

Conversation

jjasghar
Copy link
Contributor

Paul and I found this on a stream we were doing.

Signed-off-by: JJ Asghar jjasghar@gmail.com
Co-authored-by: Paul Czarkowski username.taken@gmail.com

Paul and I found this on a stream we were doing.

Signed-off-by: JJ Asghar <jjasghar@gmail.com>
Co-authored-by: Paul Czarkowski <username.taken@gmail.com>
Copy link
Owner

@alexellis alexellis left a comment

Choose a reason for hiding this comment

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

LGTM

@alexellis
Copy link
Owner

Thanks for picking that up. Can you confirm that arkade isn't affected?

@alexellis alexellis merged commit dc51804 into alexellis:master Jan 28, 2021
@alexellis
Copy link
Owner

Do you have a link to your stream?

Just a note for future PRs, "tongue in cheek" and jokes don't translate well to a git log, so perhaps something a bit more objective like "Fix YAML snippet for ClusterPullSecret"?

@jjasghar
Copy link
Contributor Author

https://youtu.be/wGVHRfeM-ZA?t=2644

@alexellis
Copy link
Owner

Thank you for the link

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

2 participants