Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Oct 4, 2019

Related to elastic/elasticsearch#46880

This PR fixes a broken link to the security content in the Stack Overview, which has moved here:
https://www.elastic.co/guide/en/elasticsearch/reference/master/mapping-roles.html#mapping-roles-file

@codecov-io
Copy link

codecov-io commented Oct 4, 2019

Codecov Report

Merging #2775 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2775   +/-   ##
=======================================
  Coverage   79.15%   79.15%           
=======================================
  Files          83       83           
  Lines        4332     4332           
=======================================
  Hits         3429     3429           
  Misses        903      903

@lcawl lcawl requested a review from bmorelli25 October 17, 2019 01:53
@lcawl lcawl marked this pull request as ready for review October 17, 2019 01:54
Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

Thanks Lisa!

@bmorelli25 bmorelli25 merged commit 3a2711b into elastic:master Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants