Skip to content

docs: add contributor ladder#301

Merged
yanmxa merged 1 commit into
genai-io:mainfrom
yanmxa:docs/contributor-ladder
Jul 13, 2026
Merged

docs: add contributor ladder#301
yanmxa merged 1 commit into
genai-io:mainfrom
yanmxa:docs/contributor-ladder

Conversation

@yanmxa

@yanmxa yanmxa commented Jul 11, 2026

Copy link
Copy Markdown
Member

Add a CONTRIBUTOR_LADDER.md — the project had an OWNERS file with reviewers/approvers but nothing documenting what those roles do or how contributors advance between them.

  • Defines four rungs — Contributor → Reviewer → Approver → Maintainer — mapped directly onto the OWNERS file.
  • Follows the CNCF contributor-ladder conventions (Responsibilities/Requirements/Privileges structure, per-role promotion process, and the standard Inactivity / Involuntary Removal / Emeritus / Contact sections).
  • Links the ladder from CONTRIBUTING.md.

@san-ci san-ci Bot added the dco:yes label Jul 11, 2026
@san-ci

san-ci Bot commented Jul 11, 2026

Copy link
Copy Markdown

[REVIEW NOTIFIER] This PR requires a review from @zhujian7.

Once this PR has been reviewed and has the lgtm label, it will be automatically squash-merged.

Suggested reviewer

Based on OWNERS, @zhujian7 is suggested to review this PR.

How to get your PR reviewed:

  • Address reviewer feedback and push changes
  • Reviewer adds lgtm label when satisfied
  • An approving review + lgtm will trigger auto-merge

@san-ci san-ci Bot added the approved label Jul 11, 2026
@yanmxa yanmxa requested review from hchenxa, ldpliu and zhujian7 July 11, 2026 15:52
@yanmxa

yanmxa commented Jul 11, 2026

Copy link
Copy Markdown
Member Author

cc @ldpliu @zhujian7 @hchenxa — this just writes down the roles our OWNERS file already implies (Contributor → Reviewer → Approver → Maintainer). The thresholds (e.g. ~3 months to Approver, 6-month inactivity) are starting points — please push back on anything that doesn't fit before we merge.

@hchenxa

hchenxa commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

@yanmxa it looks good

@yanmxa yanmxa force-pushed the docs/contributor-ladder branch from 0af45b4 to 6b7db95 Compare July 13, 2026 15:00
@san-ci san-ci Bot added the need-rebase label Jul 13, 2026
@san-ci

san-ci Bot commented Jul 13, 2026

Copy link
Copy Markdown

[REBASE NOTIFIER] Hi @yanmxa! This PR is behind the base branch and needs to be rebased.

The lgtm label has been removed and the reviewer will need to re-review after you fix this.

How to fix this

rebase your branch — Run git rebase main locally, then force-push the updated branch.

After fixing:

  • The need-rebase label will be automatically removed
  • Request a new review from the OWNERS file
  • The reviewer can add lgtm once satisfied

@yanmxa yanmxa force-pushed the docs/contributor-ladder branch from 6b7db95 to 23e55d8 Compare July 13, 2026 15:13
@san-ci san-ci Bot added need-rebase and removed need-rebase labels Jul 13, 2026
@san-ci

san-ci Bot commented Jul 13, 2026

Copy link
Copy Markdown

[REBASE NOTIFIER] Hi @yanmxa! This PR is behind the base branch and needs to be rebased.

The lgtm label has been removed and the reviewer will need to re-review after you fix this.

How to fix this

rebase your branch — Run git rebase main locally, then force-push the updated branch.

After fixing:

  • The need-rebase label will be automatically removed
  • Request a new review from the OWNERS file
  • The reviewer can add lgtm once satisfied

1 similar comment
@san-ci

san-ci Bot commented Jul 13, 2026

Copy link
Copy Markdown

[REBASE NOTIFIER] Hi @yanmxa! This PR is behind the base branch and needs to be rebased.

The lgtm label has been removed and the reviewer will need to re-review after you fix this.

How to fix this

rebase your branch — Run git rebase main locally, then force-push the updated branch.

After fixing:

  • The need-rebase label will be automatically removed
  • Request a new review from the OWNERS file
  • The reviewer can add lgtm once satisfied

Add CONTRIBUTOR_LADDER.md defining the Contributor -> Reviewer ->
Approver -> Maintainer roles mapped to the OWNERS file, following the
CNCF contributor-ladder conventions, and link it from CONTRIBUTING.md.

Signed-off-by: Meng Yan <yanmxa@gmail.com>
@yanmxa yanmxa force-pushed the docs/contributor-ladder branch from 23e55d8 to af4a7ef Compare July 13, 2026 16:05
@san-ci san-ci Bot removed the need-rebase label Jul 13, 2026
@yanmxa yanmxa added the lgtm label Jul 13, 2026
@san-ci san-ci Bot removed the lgtm label Jul 13, 2026
@san-ci

san-ci Bot commented Jul 13, 2026

Copy link
Copy Markdown

@yanmxa cannot add the lgtm label to their own PR. Another approver must review this PR and add the label.

@san-ci

san-ci Bot commented Jul 13, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR has the lgtm label but is NOT APPROVED yet.

Please request an approving review from @ldpliu (see OWNERS).

Once approved, the Review Gate will automatically squash-merge this PR.

@yanmxa yanmxa merged commit fb44518 into genai-io:main Jul 13, 2026
3 checks passed
@yanmxa yanmxa deleted the docs/contributor-ladder branch July 13, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants