Skip to content

Update strong-log-transformer version to fix warning when bootstraping#27263

Merged
mistic merged 1 commit into
elastic:masterfrom
mistic:fix-warning-caused-by-strong-log-transformer
Dec 18, 2018
Merged

Update strong-log-transformer version to fix warning when bootstraping#27263
mistic merged 1 commit into
elastic:masterfrom
mistic:fix-warning-caused-by-strong-log-transformer

Conversation

@mistic

@mistic mistic commented Dec 14, 2018

Copy link
Copy Markdown
Contributor

This is just an update of strong-log-transformer for the very last version 2.1.0.

This fixes a warning appearing on the console when bootstraping caused by a dependency node-byline that was using deprecated apis on Node 10.

More info can be found here:

strongloop/strong-log-transformer#7
jahewson/node-byline#53

@mistic mistic self-assigned this Dec 14, 2018
@mistic
mistic requested review from jbudz and joshdover December 14, 2018 22:28
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@mistic
mistic requested review from tylersmalley and removed request for jbudz December 18, 2018 03:12

@joshdover joshdover left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Shout out to @rmg for updating strong-log-transformer.

@mistic
mistic merged commit ea06d3c into elastic:master Dec 18, 2018
mistic added a commit to mistic/kibana that referenced this pull request Dec 18, 2018
mistic added a commit that referenced this pull request Dec 18, 2018
* fix(NA): update strong-log-transformer version. (#27263)

* chore(NA): merge and solve with last 6.x
@mistic

mistic commented Dec 18, 2018

Copy link
Copy Markdown
Contributor Author

6.x: 8d28976

patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
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.

3 participants