Skip to content

code clarity: replace HC with health check#5785

Merged
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
knightXun:docs
Feb 1, 2019
Merged

code clarity: replace HC with health check#5785
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
knightXun:docs

Conversation

@knightXun
Copy link
Contributor

Replace HC with health-check, make it easy understand

Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Sounds good to me, please check docs build and a nit around spelling. Thank you!

/wait

Copy link
Member

Choose a reason for hiding this comment

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

Please be consistent everywhere around "health check" (no dash, space between words). Applies elsewhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, i will do that

@mattklein123 mattklein123 self-assigned this Jan 31, 2019
@knightXun knightXun force-pushed the docs branch 4 times, most recently from 417487c to bdfbabf Compare February 1, 2019 08:32
@knightXun knightXun changed the title update docs: replace HC with health-check replace HC with health-check Feb 1, 2019
Signed-off-by: knightXun <100481562@qq.com>
@mattklein123 mattklein123 changed the title replace HC with health-check docs: replace HC with health check Feb 1, 2019
Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit fb0c648 into envoyproxy:master Feb 1, 2019
@mattklein123 mattklein123 changed the title docs: replace HC with health check code clarity: replace HC with health check Feb 1, 2019
@knightXun knightXun deleted the docs branch February 1, 2019 23:43
fredlas pushed a commit to fredlas/envoy that referenced this pull request Mar 5, 2019
Signed-off-by: knightXun <100481562@qq.com>
Signed-off-by: Fred Douglas <fredlas@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants