-
Notifications
You must be signed in to change notification settings - Fork 441
doc(v3): update top level docs #15956
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,14 +15,6 @@ intended for general use. They are subject to change and/or removal without | |
| notice. These include `google/cloud/internal/`, and | ||
| `google/cloud/testing_utils/`. | ||
|
|
||
| ## Supported Platforms | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. qq: Why is this section being removed? Is 3.x compatible with all common platforms?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That same information already exists in the top level README.md. It was duplicated here and didn't seem necessary to keep both. |
||
|
|
||
| - Windows, macOS, Linux | ||
| - C++14 (and higher) compilers. We test with GCC >= 7.5, Clang >= 6.0, and MSVC | ||
| \>= 2022 | ||
| - Environments with or without exceptions | ||
| - Bazel (>= 6.0) and CMake (>= 3.5) builds | ||
|
|
||
| ## Documentation | ||
|
|
||
| - [Reference doxygen documentation][doxygen-link] for each release of this | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With OpenTelemetry now being a required dependency, the referenced footnote
[^7]is outdated as it still refers to it as an 'optional dependency'. Suggest removing the reference to avoid confusion.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated footnote.