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

Update PROJECT_NAME_incubation_stage.md #276

Merged
merged 8 commits into from
Mar 18, 2024
Merged

Conversation

SecurityCRob
Copy link
Contributor

proposed changes to incubating project template

Copy link
Contributor

@marcelamelara marcelamelara left a comment

Choose a reason for hiding this comment

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

Just one point of clarification.

process/templates/PROJECT_NAME_incubation_stage.md Outdated Show resolved Hide resolved
process/templates/PROJECT_NAME_incubation_stage.md Outdated Show resolved Hide resolved
process/templates/PROJECT_NAME_incubation_stage.md Outdated Show resolved Hide resolved
process/templates/PROJECT_NAME_incubation_stage.md Outdated Show resolved Hide resolved
Comment on lines +31 to +32
Project is integrated into the OpenSSF Scorecard
* "link to Scorecard output"
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this is a requirement today, but I think it's reasonable if we want to add it.

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.

What a process it is rationalizing our lifecycle docs, TI gives and gets, and these templates; thanks for putting this together!

Copy link
Contributor

@torgo torgo left a comment

Choose a reason for hiding this comment

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

lgtm

Adding a requirement for project leadership

Signed-off-by: Jay White <[email protected]>
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.

LGTM!

@SecurityCRob
Copy link
Contributor Author

6 of 9 TAC members approve, merging

@SecurityCRob SecurityCRob merged commit 9ddc1bb into main Mar 18, 2024
6 of 7 checks passed
@SecurityCRob SecurityCRob deleted the SecurityCRob-patch-4 branch March 18, 2024 18:04
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.

8 participants