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

What: GB mtg - Add competencies and DD throughput #1499

Merged
merged 7 commits into from
Dec 9, 2024

Conversation

TheFoxAtWork
Copy link
Contributor

Why:

  • it was requested the TOC describe used competencies for their work to better inform candidates
  • it was requested the TOC explicitly state minimum throughput for Due Diligence

This change address the need by:

  • Adding explicit throughput on line 39 for accountability
  • Establish initial competencies at line 47

Why:
 * it was requested the TOC describe used competencies for their work to better inform candidates
 * it was requested the TOC explicitly state minimum throughput for Due Diligence

This change address the need by:
 * Adding explicit throughput on line 39 for accountability
 * Establish initial competencies at line 47

Signed-off-by: Emily Fox <[email protected]>
Copy link
Contributor

@linsun linsun left a comment

Choose a reason for hiding this comment

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

lgtm

@craigbox
Copy link
Contributor

craigbox commented Dec 6, 2024

Nit: I think the "onboarding" title/theme of this doc is somewhat overloaded, and suggest you might like to break some or all of this out into an "expectations" doc.

Copy link
Member

@cblecker cblecker left a comment

Choose a reason for hiding this comment

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

+1 (non-binding)

Copy link
Member

@dims dims left a comment

Choose a reason for hiding this comment

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

LGTM

@alena1108
Copy link
Contributor

+1 (non-binding)

Copy link
Contributor

@angellk angellk left a comment

Choose a reason for hiding this comment

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

@TheFoxAtWork added some comments where it would be good to clarify further

jberkus and others added 4 commits December 9, 2024 14:52
Signed-off-by: Ricardo Aravena <[email protected]>

Signed-off-by: Ricardo Aravena <[email protected]>
@TheFoxAtWork
Copy link
Contributor Author

@angellk We'll need to rework the TOC DD guide to include the more recent changes, like the lightweight/triage portions. I've added the indicator that it is expected, but we dont have info on what that actually is: make sure the adopter form is filled out, make sure the criteria are initially met, what to do when a project isnt ready ,etc.

Copy link
Contributor

@angellk angellk left a comment

Choose a reason for hiding this comment

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

thank you @TheFoxAtWork ! i'll open the corresponding issue for the DD guide updates - and with 4 TOC reviewers and 2 GB reviewers, looks good to merge.

@angellk angellk merged commit 993f577 into cncf:main Dec 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.