Skip to content

Conversation

@graphaelli
Copy link
Member

No description provided.

@graphaelli graphaelli requested a review from simitt June 12, 2018 16:16
@graphaelli graphaelli changed the title Approval newline Add trailing new lines in generated files Jun 12, 2018
Copy link
Contributor

@simitt simitt left a comment

Choose a reason for hiding this comment

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

LGTM - just out of curiosity, is there a specific reason for this?

@graphaelli
Copy link
Member Author

no specific reason

@graphaelli graphaelli merged commit 1c1edf7 into elastic:master Jun 13, 2018
@graphaelli graphaelli deleted the approval-newline branch June 13, 2018 15:58
simitt pushed a commit to simitt/apm-server that referenced this pull request Jul 4, 2018
* add trailing newline to approvals

* add trailing newline to elasticsearch docs
@simitt simitt mentioned this pull request Jul 4, 2018
simitt pushed a commit to simitt/apm-server that referenced this pull request Jul 4, 2018
* add trailing newline to approvals

* add trailing newline to elasticsearch docs
simitt pushed a commit that referenced this pull request Jul 4, 2018
* add trailing newline to approvals

* add trailing newline to elasticsearch docs
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