Skip to content

Latest commit

 

History

History
101 lines (69 loc) · 5.5 KB

CONTRIBUTING.md

File metadata and controls

101 lines (69 loc) · 5.5 KB
Raw

Contributing to opentelemetry-go-contrib

Welcome to the OpenTelemetry Go contrib repository! Thank you for your interest in contributing to this project. Before you start please be sure to read through these contributing requirements and recommendations.

Become a Contributor

All contributions to this project MUST be licensed under this project's license. You will need to sign the CNCF CLA before your contributions will be accepted.

Filing Issues

Sensitive security-related issues should be reported to cncf-opentelemetry-tc@lists.cncf.io. See the security policy for details.

When reporting bugs, please be sure to include the following.

  • What version of Go and opentelemetry-go-contrib are you using?
  • What operating system and processor architecture are you using?
  • What did you do?
  • What did you expect to see?
  • What did you see instead?

For instrumentation requests, please see the instrumentation documentation.

Contributing Code

The project welcomes code patches, but to make sure things are well coordinated you should discuss any significant change before starting the work. It's recommended that you signal your intention to contribute in the issue tracker, either by filing a new issue or by claiming an existing one.

Style Guide

  • Code should conform to the opentelemetry-go Style Guide.
  • Make sure to run make precommit - this will find and fix issues with the code formatting.

Pull Requests

All pull requests need to be made from a fork of this repository. Changes should be made using the GitHub flow and submitted as a pull request to this repository.

A pull request is considered ready to merge when the following criteria are meet.

  • It has received two approvals from Approvers/Maintainers (at different companies), unless the change is for an exempt module1.
  • All feedback has been addressed. Be sure to "Resolve" all comments that have been addressed to signal this.
  • Any substantive changes submitted after an Approval removes that Approval. You will need to manually clear these prior Approval reviews to indicate to the reviewer that they need to resubmit their review. This includes changes resulting from other feedback. Unless the approver explicitly stated that their approval will persist across changes it should be assumed that the pull request needs their review again. Other project members (e.g. approvers, maintainers) can help with this if there are any questions or if you forget to clear reviews.
  • If the changes are not trivial, cosmetic, exempt1, or for documentation or dependencies only, the pull request will need to be open for review for at least one working day. This gives people reasonable time to review.
  • CHANGELOG.md has been updated to reflect what has been added, changed, removed, or fixed from the end users perspective. See how to keep a changelog.
  • Urgent fixes can take exception as long as it has been actively communicated.

Any Maintainer can merge the pull request once it is ready to merge.

Draft Pull Requests

It can be helpful at times to publish your incomplete changes. To do this create a draft pull request. Additionally, you can prefix the pull request title with [WIP].

Where to Get Help

You can connect with us in our slack channel.

The Go special interest group (SIG) meets regularly. See the OpenTelemetry community repo for information on this and other language SIGs. See the public meeting notes for a summary description of past meetings.

Approvers and Maintainers

Approvers:

Maintainers:

Emeritus:

Become an Approver or a Maintainer

See the community membership document in OpenTelemetry community repo.

Footnotes

  1. The go.opentelemetry.io/contrib/instrgen module is exempt from the two approvals and one day requirement. Only one approval is needed to merge a Pull Request for that module and there is no minimum amout of time required for the PR to be open before merging. This exemption is to be removed when that package makes its first tagged release. ↩ ↩2