Skip to content

[Lens] remove warnings when running the tests#131373

Merged
mbondyra merged 1 commit intoelastic:mainfrom
mbondyra:lens/partition_vis_test_warnings
May 3, 2022
Merged

[Lens] remove warnings when running the tests#131373
mbondyra merged 1 commit intoelastic:mainfrom
mbondyra:lens/partition_vis_test_warnings

Conversation

@mbondyra
Copy link
Copy Markdown
Contributor

@mbondyra mbondyra commented May 3, 2022

Summary

When running the tests from the corrected file, there's a lot of warnings showing (no need to test, you can check it in CI output). This corrects it.

Screenshot 2022-05-03 at 10 10 28

@mbondyra mbondyra added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v8.3.0 labels May 3, 2022
@mbondyra mbondyra requested a review from a team as a code owner May 3, 2022 08:15
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors)

Copy link
Copy Markdown
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

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

LGTM, code review only

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@mbondyra mbondyra merged commit 0558903 into elastic:main May 3, 2022
kertal pushed a commit to kertal/kibana that referenced this pull request May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants