Skip to content

Upgrade @elastic/charts to 69.2.1#216430

Merged
markov00 merged 3 commits intoelastic:mainfrom
markov00:2025_03_31-upgrade_elastic-charts
Apr 1, 2025
Merged

Upgrade @elastic/charts to 69.2.1#216430
markov00 merged 3 commits intoelastic:mainfrom
markov00:2025_03_31-upgrade_elastic-charts

Conversation

@markov00
Copy link
Contributor

@markov00 markov00 commented Mar 31, 2025

Summary

This PR updates @elastic/charts library to 69.2.1, that reverse the wrongly merged multi-time axis breaking change and keeps only the new feature added to Metric about exposing the fontsize and the color styling

v69.2.0
Compare Source

Features
metric: Expose both font size and color stylings on the extra prop (#​2627) (2d21118)

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #5 / should render all components

Metrics [docs]

Page load bundle

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

id before after diff
kbnUiSharedDeps-npmDll 6.1MB 6.1MB -3.9KB

History

@markov00 markov00 marked this pull request as ready for review April 1, 2025 07:30
@markov00 markov00 requested a review from a team as a code owner April 1, 2025 07:30
@markov00 markov00 added release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels v9.1.0 v8.19.0 Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// dependencies Pull requests that update a dependency file labels Apr 1, 2025
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@dej611 dej611 left a comment

Choose a reason for hiding this comment

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

I really like this release.

@markov00 markov00 merged commit f8b3e93 into elastic:main Apr 1, 2025
22 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/14192183886

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 216430

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 3, 2025
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 216430 locally

@markov00
Copy link
Contributor Author

markov00 commented Apr 4, 2025

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

markov00 added a commit to markov00/kibana that referenced this pull request Apr 4, 2025
## Summary

This PR updates `@elastic/charts` library to 69.2.1, that reverse the
wrongly merged multi-time axis breaking change and keeps only the new
feature added to Metric about exposing the fontsize and the color
styling

[v69.2.0](https://github.com/elastic/elastic-charts/blob/HEAD/CHANGELOG.md#6920-2025-03-26)
[Compare
Source](https://github.com/elastic/elastic-charts/compare/v69.1.1...v69.2.0)

Features
metric: Expose both font size and color stylings on the extra prop
([#​2627](https://github.com/elastic/elastic-charts/issues/2627))
([2d21118](https://github.com/elastic/elastic-charts/commit/2d2111899588d6619dbe4152fd8113cd2215fa1a))

(cherry picked from commit f8b3e93)

# Conflicts:
#	package.json
#	yarn.lock
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

2 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@markov00 markov00 removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 9, 2025
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 10, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

2 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

3 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

2 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@elastic elastic deleted a comment from kibanamachine Apr 29, 2025
@elastic elastic deleted a comment from kibanamachine Apr 29, 2025
@elastic elastic deleted a comment from kibanamachine Apr 29, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

3 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

1 similar comment
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @markov00

markov00 added a commit that referenced this pull request May 9, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [Upgrade @elastic/charts to 69.2.1
(#216430)](#216430)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Marco
Vettorello","email":"marco.vettorello@elastic.co"},"sourceCommit":{"committedDate":"2025-04-01T09:31:12Z","message":"Upgrade
@elastic/charts to 69.2.1 (#216430)\n\n## Summary\n\nThis PR updates
`@elastic/charts` library to 69.2.1, that reverse the\nwrongly merged
multi-time axis breaking change and keeps only the new\nfeature added to
Metric about exposing the fontsize and the
color\nstyling\n\n\n[v69.2.0](https://github.com/elastic/elastic-charts/blob/HEAD/CHANGELOG.md#6920-2025-03-26)\n[Compare\nSource](https://github.com/elastic/elastic-charts/compare/v69.1.1...v69.2.0)\n\nFeatures\nmetric:
Expose both font size and color stylings on the extra
prop\n([#​2627](https://github.com/elastic/elastic-charts/issues/2627))\n([2d21118](https://github.com/elastic/elastic-charts/commit/2d2111899588d6619dbe4152fd8113cd2215fa1a))","sha":"f8b3e935730dbe2e17f57c39d089e7b04f1a7bd4","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Visualizations","release_note:skip","dependencies","backport
missing","backport:version","v9.1.0","v8.19.0"],"title":"Upgrade
@elastic/charts to
69.2.1","number":216430,"url":"https://github.com/elastic/kibana/pull/216430","mergeCommit":{"message":"Upgrade
@elastic/charts to 69.2.1 (#216430)\n\n## Summary\n\nThis PR updates
`@elastic/charts` library to 69.2.1, that reverse the\nwrongly merged
multi-time axis breaking change and keeps only the new\nfeature added to
Metric about exposing the fontsize and the
color\nstyling\n\n\n[v69.2.0](https://github.com/elastic/elastic-charts/blob/HEAD/CHANGELOG.md#6920-2025-03-26)\n[Compare\nSource](https://github.com/elastic/elastic-charts/compare/v69.1.1...v69.2.0)\n\nFeatures\nmetric:
Expose both font size and color stylings on the extra
prop\n([#​2627](https://github.com/elastic/elastic-charts/issues/2627))\n([2d21118](https://github.com/elastic/elastic-charts/commit/2d2111899588d6619dbe4152fd8113cd2215fa1a))","sha":"f8b3e935730dbe2e17f57c39d089e7b04f1a7bd4"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/216430","number":216430,"mergeCommit":{"message":"Upgrade
@elastic/charts to 69.2.1 (#216430)\n\n## Summary\n\nThis PR updates
`@elastic/charts` library to 69.2.1, that reverse the\nwrongly merged
multi-time axis breaking change and keeps only the new\nfeature added to
Metric about exposing the fontsize and the
color\nstyling\n\n\n[v69.2.0](https://github.com/elastic/elastic-charts/blob/HEAD/CHANGELOG.md#6920-2025-03-26)\n[Compare\nSource](https://github.com/elastic/elastic-charts/compare/v69.1.1...v69.2.0)\n\nFeatures\nmetric:
Expose both font size and color stylings on the extra
prop\n([#​2627](https://github.com/elastic/elastic-charts/issues/2627))\n([2d21118](https://github.com/elastic/elastic-charts/commit/2d2111899588d6619dbe4152fd8113cd2215fa1a))","sha":"f8b3e935730dbe2e17f57c39d089e7b04f1a7bd4"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Nick Partridge <nicholas.partridge@elastic.co>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels dependencies Pull requests that update a dependency file 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.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants