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.

Sign up for GitHub

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

Jump to bottom

docs: creates an initial SECURITY.md policy #60

Merged
merged 4 commits into from
Aug 30, 2024
Merged

docs: creates an initial SECURITY.md policy #60

merged 4 commits into from
Aug 30, 2024

Conversation

jpower432
Copy link
Member

What

This creates an initial security policy and process in the SECUIRTY.md .

Closes #4

Impacts

  • Actions by admins following this merge - follow these steps for each project

@jpower432 jpower432 added documentation Improvements or additions to documentation governance Project governance labels Aug 6, 2024
jflowers
jflowers approved these changes Aug 7, 2024
View reviewed changes
Copy link
Member

@jflowers jflowers left a comment

Choose a reason for hiding this comment

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

LGTM

jflowers
jflowers approved these changes Aug 7, 2024
View reviewed changes
Copy link
Member

@jflowers jflowers left a comment

Choose a reason for hiding this comment

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

Good catch

@jpower432 jpower432 requested a review from ancatri August 27, 2024 17:07
@yuji-watanabe-jp
Copy link
Member

LGTM
+1

vikas-agarwal76
vikas-agarwal76 approved these changes Aug 28, 2024
View reviewed changes
Copy link
Contributor

@vikas-agarwal76 vikas-agarwal76 left a comment

Choose a reason for hiding this comment

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

LGTM

degenaro
degenaro approved these changes Aug 30, 2024
View reviewed changes
Copy link
Contributor

@degenaro degenaro left a comment

Choose a reason for hiding this comment

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

LGTM

@jpower432
Copy link
Member Author

Merging this with 2/3 of the reviewer approval.

@jpower432 jpower432 merged commit 3e5fde4 into main Aug 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vikas-agarwal76 vikas-agarwal76 vikas-agarwal76 approved these changes

@degenaro degenaro degenaro approved these changes

@jflowers jflowers jflowers approved these changes

@mrgadgil mrgadgil Awaiting requested review from mrgadgil

@yuji-watanabe-jp yuji-watanabe-jp Awaiting requested review from yuji-watanabe-jp

@ancatri ancatri Awaiting requested review from ancatri

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation governance Project governance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add an org security policy
5 participants
@jpower432 @yuji-watanabe-jp @jflowers @degenaro @vikas-agarwal76