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

updating CoC link and adding security section #211

Closed
wants to merge 2 commits into from
Closed

updating CoC link and adding security section #211

wants to merge 2 commits into from

Conversation

vsoch
Copy link

@vsoch vsoch commented Feb 26, 2019

Signed-off-by: Vanessa Sochat [email protected]

opencontainers/.github#3

@cyphar
Copy link
Member

cyphar commented Feb 26, 2019

Please squash your commits.

@vsoch
Copy link
Author

vsoch commented Feb 27, 2019

@cyphar what about this one - where I've done the PR from master? Is the only way to start afresh?

@vsoch
Copy link
Author

vsoch commented Feb 27, 2019

I'm getting a "noop"

image

@cyphar
Copy link
Member

cyphar commented Feb 27, 2019

You need to do git rebase -i <what it was based on>. So this would be git rebase -i master or git rebase -i origin/master.

@vsoch
Copy link
Author

vsoch commented Feb 27, 2019

I didn't check out a new branch here, I commit to master, so I'll just do this one over.

@vsoch
Copy link
Author

vsoch commented Feb 27, 2019

This is now a ghost pull request - the tree it comes from is nothing more than a memory!

@vsoch vsoch closed this Feb 27, 2019
@cyphar
Copy link
Member

cyphar commented Feb 27, 2019

Sure -- for the record you could've done it with git rebase -i origin/master or by passing the commit ID instead with git rebase -i $commit.

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.

2 participants