open-telemetry / open-telemetry/opentelemetry-cpp

Revisit the merge policy for the repo.

Open
#2,092 1 comment 0 reactions 1 assignee View on GitHub

@lalitb is already working on this.

Since Apr 12, 2023.

do-not-stale
Dominant language
C++
Stars
1.4k
Forks
632
Avg merge
1d 13h
Merged PRs (30d)
75

Description

Revisit merge policy and get more clarity in terms of

  • Size of the PR
  • Number of approvals required based on size of the PR
  • Define days for PR review based on the size of the PR
  • Escalation process if the PR is not reviewed in given days.

Please add any more details if required to bring more clarity.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.