Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependency @elastic/charts to v37 #5243

Merged
merged 2 commits into from
Oct 6, 2021
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Oct 5, 2021

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@elastic/charts ^36.0.0 -> ^37.0.0 age adoption passing confidence

Release Notes

elastic/elastic-charts

v37.0.0

Compare Source

Bug Fixes
Code Refactoring
BREAKING CHANGES
  • DEFAULT_CHART_MARGINS, DEFAULT_GEOMETRY_STYLES, DEFAULT_CHART_PADDING and DEFAULT_MISSING_COLOR are no longer exposed as part of the API
  • The public type varieties for domains are discontinued, in favor of retaining the single DomainRange export, which now has a mandatory {min: number, max: number}. The developer can supply NaN where a finite min, max or both aren't defined (ie. in place of former effective undefined). In addition, some console.warn punctuations changed

Co-authored-by: Marco Vettorello [email protected]
Co-authored-by: Nick Partridge [email protected]


Configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box.

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot added the dependencies Pull requests that update a dependency file label Oct 5, 2021
@kibanamachine
Copy link

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cee-chen
Copy link
Contributor

cee-chen commented Oct 5, 2021

jenkins test this

@cchaos
Copy link
Contributor

cchaos commented Oct 5, 2021

Looks like what wasn't accounted for in their breaking changes section is the removal of the textInvertible theme key. I'll try to get to it later today, unless someone gets to it first.

@renovate renovate bot force-pushed the renovate/@elasticcharts branch 2 times, most recently from 059cbbe to 9bf120a Compare October 5, 2021 21:44
@renovate renovate bot force-pushed the renovate/@elasticcharts branch from 9bf120a to 001d9b5 Compare October 5, 2021 23:39
@cchaos cchaos enabled auto-merge (squash) October 6, 2021 00:01
@cchaos
Copy link
Contributor

cchaos commented Oct 6, 2021

jenkins, test this

@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_5243/

@cchaos cchaos merged commit 3d9a593 into master Oct 6, 2021
@renovate renovate bot deleted the renovate/@elasticcharts branch October 6, 2021 15:59
ym pushed a commit to ym/eui that referenced this pull request Oct 29, 2021
* Remove outdated theme keys

Co-authored-by: Renovate Bot <[email protected]>
Co-authored-by: cchaos <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants