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

adding security and CoC links #58

Closed
wants to merge 1 commit into from
Closed

adding security and CoC links #58

wants to merge 1 commit into from

Conversation

vsoch
Copy link

@vsoch vsoch commented Feb 26, 2019

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

opencontainers/.github#3

@vbatts
Copy link
Member

vbatts commented Feb 26, 2019

LGTM

Approved with PullApprove Approved with PullApprove Approved with PullApprove Approved with PullApprove Approved with PullApprove

@cyphar
Copy link
Member

cyphar commented Feb 27, 2019

o.O

2019-02-27-115555_593x70_scrot

@cyphar
Copy link
Member

cyphar commented Feb 27, 2019

LGTM.

Approved with PullApprove Approved with PullApprove Approved with PullApprove

@vbatts
Copy link
Member

vbatts commented Feb 27, 2019

LGTM

Approved with PullApprove Approved with PullApprove Approved with PullApprove Approved with PullApprove Approved with PullApprove

@vbatts
Copy link
Member

vbatts commented Feb 27, 2019

omg, pullapprove is confus
image

It's requiring approval of all the groups ? @caniszczyk have you seen that before?

@cyphar
Copy link
Member

cyphar commented Feb 27, 2019

I think there's some weird configuration in place for project-template but it's just ridiculous to require 2 LGTMs from every group -- especially some groups like @opencontainers/selinux-maintainers which only has 3 people in it.

@vbatts
Copy link
Member

vbatts commented Feb 27, 2019 via email

Signed-off-by: Vanessa Sochat <[email protected]>
@vsoch
Copy link
Author

vsoch commented Feb 28, 2019

Code of conduct link is updated to be outside of .github (and in all caps) to reflect OCI call discussion.

@vbatts
Copy link
Member

vbatts commented Feb 28, 2019

LGTM

Approved with PullApprove Approved with PullApprove Approved with PullApprove Approved with PullApprove Approved with PullApprove

@vsoch vsoch closed this by deleting the head repository Apr 29, 2023
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