Skip to content
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

Improve bootstrap checks error messages #24548

Merged
merged 1 commit into from May 8, 2017

Conversation

Projects
None yet
3 participants
@jasontedor
Copy link
Member

commented May 8, 2017

When multiple bootstrap checks fail, it's not clear where one error message begins and the next error message ends. This commit numbers the bootstrap check error messages so they are easier to read.

Improve bootstrap checks error messages
When multiple bootstrap checks fail, it's not clear where one error
message begins and the next error message ends. This commit numbers the
bootstrap check error messages so they are easier to read.
@jaymode

jaymode approved these changes May 8, 2017

Copy link
Member

left a comment

LGTM

@jasontedor jasontedor merged commit bf32b0c into elastic:master May 8, 2017

2 checks passed

CLA Commit author is a member of Elasticsearch
Details
elasticsearch-ci Build finished.
Details

jasontedor added a commit that referenced this pull request May 8, 2017

Improve bootstrap checks error messages
When multiple bootstrap checks fail, it's not clear where one error
message begins and the next error message ends. This commit numbers the
bootstrap check error messages so they are easier to read.

Relates #24548

jasontedor added a commit that referenced this pull request May 8, 2017

Improve bootstrap checks error messages
When multiple bootstrap checks fail, it's not clear where one error
message begins and the next error message ends. This commit numbers the
bootstrap check error messages so they are easier to read.

Relates #24548

@jasontedor jasontedor deleted the jasontedor:bootstrap-checks-formatting branch May 8, 2017

@jasontedor

This comment has been minimized.

Copy link
Member Author

commented May 8, 2017

@clintongormley clintongormley added v6.0.0-alpha2 and removed v6.0.0 labels Jun 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.