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

docs: simplify contrib part in readme #2814

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

Flarna
Copy link
Member

@Flarna Flarna commented Mar 3, 2022

Which problem is this PR solving?

Short description of the changes

Removed the contrib version column from compatibility matrix because contrib packages
no longer use a common version and release happens quite independent of core.
Therefore having this at a single place in this repo would result in mostly
incomplete/outdated information.

Removed links to individual contrib packages as the list was outdated already and it's
a maintenance pain to keep core repo up to date with unrelated contrib changes.

Type of change

  • This change requires a documentation update

Removed the contrib version column from compatibility matrix because contrib packages
no longer use a common version and release happens quite independent of core.
Therefore having this at a single place in this repo would result in mostly
incomplete/outdated information.

Removed links to individual contrib packages as the list was outdated already and it's
a maintenance pain to keep core repo up to date with unrelated contrib changes.
@Flarna Flarna requested a review from a team March 3, 2022 07:16
@codecov
Copy link

codecov bot commented Mar 3, 2022

Codecov Report

Merging #2814 (4f4c8d0) into main (df0cee9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2814   +/-   ##
=======================================
  Coverage   93.39%   93.39%           
=======================================
  Files         159      159           
  Lines        5450     5450           
  Branches     1145     1145           
=======================================
  Hits         5090     5090           
  Misses        360      360           

@dyladan dyladan added the document Documentation-related label Mar 3, 2022
@dyladan dyladan changed the title chore: simplify contrib part in readme docs: simplify contrib part in readme Mar 3, 2022
@dyladan dyladan merged commit ef03d02 into open-telemetry:main Mar 3, 2022
@Flarna Flarna deleted the update-contrib branch March 3, 2022 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document Documentation-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants