Skip to content

[TSVB] Change allowJs setting to true#147176

Merged
stratoula merged 6 commits intoelastic:mainfrom
stratoula:tsvb-allowJs
Dec 13, 2022
Merged

[TSVB] Change allowJs setting to true#147176
stratoula merged 6 commits intoelastic:mainfrom
stratoula:tsvb-allowJs

Conversation

@stratoula
Copy link
Contributor

@stratoula stratoula commented Dec 7, 2022

Summary

Closes #144284

This PR sets the allowJs to true and solves the TS problems.

@stratoula stratoula added Feature:TSVB TSVB (Time Series Visual Builder) release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v8.7.0 labels Dec 8, 2022
@stratoula stratoula marked this pull request as ready for review December 8, 2022 12:15
@stratoula stratoula requested a review from a team as a code owner December 8, 2022 12:15
@stratoula stratoula added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Dec 8, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations)

@stratoula stratoula changed the title Change allowJs setting to true [TSVB] Change allowJs setting to true Dec 8, 2022
…rocessors/table/sibling_buckets.ts

Co-authored-by: Marco Liberati <dej611@users.noreply.github.com>
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
visTypeTimeseries 511.4KB 511.4KB +4.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
visTypeTimeseries 19.7KB 19.9KB +228.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 60 66 +6
osquery 109 115 +6
securitySolution 445 451 +6
total +20

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 69 75 +6
osquery 110 117 +7
securitySolution 521 527 +6
total +21

History

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

@stratoula
Copy link
Contributor Author

@dej611 do you want any other change?

@stratoula stratoula merged commit 9db5f8f into elastic:main Dec 13, 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 Feature:TSVB TSVB (Time Series Visual Builder) 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.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[vis_types/timeseries] Enable allowJs and fix type issues

4 participants