Skip to content

[Style] Fix line length violations for threadpool, indexing, and script packages#37205

Merged
jdconrad merged 10 commits intoelastic:masterfrom
jdconrad:line0
Jan 9, 2019
Merged

[Style] Fix line length violations for threadpool, indexing, and script packages#37205
jdconrad merged 10 commits intoelastic:masterfrom
jdconrad:line0

Conversation

@jdconrad
Copy link
Copy Markdown
Contributor

@jdconrad jdconrad commented Jan 7, 2019

This fixes line length violations in several files across the threadpool, indexing, and script packages.

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

Copy link
Copy Markdown
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@jdconrad jdconrad removed the request for review from danielmitterdorfer January 7, 2019 21:36
@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 7, 2019

@rjernst Thanks for the review! Will commit as soon as CI completes.

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine test this please

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine test this please

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine run gradle build tests 2

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine run gradle build tests 1

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine run gradle build tests 2

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine test this please

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 8, 2019

@elasticmachine run gradle build tests 1

1 similar comment
@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 1

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 2

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 1

1 similar comment
@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 1

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 2

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 1

@jdconrad
Copy link
Copy Markdown
Contributor Author

jdconrad commented Jan 9, 2019

@elasticmachine run gradle build tests 1

@jdconrad jdconrad merged commit 95eef77 into elastic:master Jan 9, 2019
jdconrad added a commit to jdconrad/elasticsearch that referenced this pull request Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants