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

fix(readme): update compatibility table and remove API column #4104

Merged
merged 9 commits into from
Sep 14, 2023

Conversation

pichlermarc
Copy link
Member

@pichlermarc pichlermarc commented Aug 31, 2023

Which problem is this PR solving?

The table needed to be updated, and API version requirements seemed to be more restrictive than they actually were. This PR removes the API column and instead adds notes to the SDK versions that require newer API versions.

Fixes #4094

@pichlermarc pichlermarc added document Documentation-related Skip Changelog labels Aug 31, 2023
@pichlermarc pichlermarc requested a review from a team August 31, 2023 13:06
@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Merging #4104 (0468856) into main (f6ebf0e) will increase coverage by 0.32%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head 0468856 differs from pull request most recent head 52de9f1. Consider uploading reports for the commit 52de9f1 to get more accurate results

@@            Coverage Diff             @@
##             main    #4104      +/-   ##
==========================================
+ Coverage   92.23%   92.56%   +0.32%     
==========================================
  Files         326      292      -34     
  Lines        9299     8178    -1121     
  Branches     1971     1692     -279     
==========================================
- Hits         8577     7570    -1007     
+ Misses        722      608     -114     

README.md Outdated Show resolved Hide resolved
@pichlermarc pichlermarc merged commit 937cfaa into open-telemetry:main Sep 14, 2023
16 checks passed
@pichlermarc pichlermarc deleted the fix/table branch September 14, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document Documentation-related Skip Changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

README.md needs clarification re Package Version Compatibility
4 participants