-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Conversation
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.
LGTM⛴ but some comments to simplify even more doc updates during releases
apm-server/README.md
Outdated
@@ -8,8 +8,8 @@ The design and code is less mature than official GA features and is being | |||
provided as-is with no warranties. Alpha features are not subject to the support | |||
SLA of official GA features (see [supported configurations][] for more details). | |||
|
|||
**Warning**: This branch is used for development, please use [7.7.1][] release | |||
for released version. | |||
**Warning**: This branch is used for development, please use the latest [7.10](https://github.com/elastic/helm-charts/releases) |
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.
7.10 isn't released yet so this is not accurate. Also this would require us to update it each time we would release a new minor version (7.11, ...) to stay up to date.
Maybe we could just mention the latest 7.x release
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.
same comment as #859 (comment) about having the warning statement in a single line
jenkins test this please |
Reword warning statements in chart readme to streamline the release process - related PR : #858, #859