Skip to content

Changed securtityContext to securityContext #7374

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

Merged
1 commit merged into from Feb 2, 2018
Merged

Changed securtityContext to securityContext #7374

1 commit merged into from Feb 2, 2018

Conversation

ghost
Copy link

@ghost ghost commented Jan 29, 2018

@jamesread Please review. This is a follow-up to #7351.

@ghost ghost added this to the Next Release milestone Jan 29, 2018
@ghost ghost self-assigned this Jan 29, 2018
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 29, 2018
@ghost ghost mentioned this pull request Jan 29, 2018
@jamesread
Copy link

LGTM :)

@ghost
Copy link
Author

ghost commented Feb 1, 2018

@openshift/team-documentation PTAL. No rev history needed.

@mburke5678
Copy link
Contributor

@tmorriso-rh LGTM

@jamesread
Copy link

Hey @tmorriso-rh , first time contributor here (if you can call a typo commit a contribution!), just curious why you created your own branch again rather than merging from mine? It is because I didn't apply the right labels to the PR or something or do you prefer not to merge from people outside the project or something? Not fussed, just curious. Thanks!

@ghost
Copy link
Author

ghost commented Feb 2, 2018

Hi @jamesread This was actually a mistake on my part. I should have fetched your PR, which would have allowed me to view and approve your contribution, and then merge the PR. So, no, you did things correctly. :-) Sorry for the confusion.

@ghost ghost added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 2, 2018
@ghost ghost merged commit 557cfee into openshift:master Feb 2, 2018
@vikram-redhat vikram-redhat modified the milestones: Next Release, Staging, Published - 02/07/2018 Feb 6, 2018
@ghost ghost deleted the PR-7351-fix-typo branch February 13, 2018 17:15
This pull request was closed.
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.

4 participants