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

policy: add github org membership section #171

Merged
merged 1 commit into from
Jun 29, 2023
Merged

policy: add github org membership section #171

merged 1 commit into from
Jun 29, 2023

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented May 31, 2023

Followup from #155 and #155 (comment)

There are currently no dangerous permissions inherently granted by being a member of the github org, and I don't foresee anyone changing that (at all, let alone without broad discussion and TAC approval).

Note: the wording here is "should", intentionally - this paragraph in no way obligates OpenSSF to add anyone to the github org. However, once someone is in the org, they can be freely added to and removed from teams - prior to being in the org, they have to accept an invitation that expires in 7 days, so it's harder to use teams as a self-documenting structure when you have to wait on invitation acceptance.

@ljharb ljharb requested review from steiza and a team May 31, 2023 18:00
Copy link
Contributor

@SecurityCRob SecurityCRob left a comment

Choose a reason for hiding this comment

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

+1

@ljharb ljharb requested a review from a team June 14, 2023 16:45
@ljharb ljharb requested a review from a team June 14, 2023 17:17
Copy link
Contributor

@lehors lehors left a comment

Choose a reason for hiding this comment

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

Even though it isn't very relevant in this case could you please add the signoff to fix the DCO issue?

@ljharb
Copy link
Member Author

ljharb commented Jun 14, 2023

@lehors whoops, thanks, done

Copy link
Member

@steiza steiza left a comment

Choose a reason for hiding this comment

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

Looks good to me; thanks!

@ljharb
Copy link
Member Author

ljharb commented Jun 29, 2023

I plan to merge this tomorrow if there’s no objections.

policies/access.md Outdated Show resolved Hide resolved
@ljharb ljharb merged commit a4ae081 into main Jun 29, 2023
@ljharb ljharb deleted the policies branch June 29, 2023 16:26
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.

6 participants