Skip to content

Conversation

@lcawl lcawl changed the title [DOCS] Indicate Apache and Nginx ML modules are legacy versions [DOCS] Update list of Apache ML modules Jun 3, 2021
@lcawl lcawl added the v7.13.2 label Jun 3, 2021
@lcawl lcawl marked this pull request as ready for review June 3, 2021 21:35
@lcawl lcawl requested review from peteharverson and szabosteve June 3, 2021 21:35
@lcawl lcawl changed the title [DOCS] Update list of Apache ML modules [DOCS] Update list of Apache and Nginx ML modules Jun 3, 2021
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the list! I have two minor comments. Otherwise, LGTM!

lcawl and others added 3 commits June 4, 2021 08:18
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Copy link

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Nginx text needs a couple of edits.

Copy link

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest edits LGTM

@lcawl lcawl merged commit 9c12937 into elastic:master Jun 9, 2021
@lcawl lcawl deleted the legacy-modules branch June 9, 2021 19:23
lcawl added a commit to lcawl/stack-docs that referenced this pull request Jun 9, 2021
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
lcawl added a commit to lcawl/stack-docs that referenced this pull request Jun 9, 2021
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment