Skip to content

Align chroma-js with Kibana#1095

Merged
jsanz merged 1 commit intoelastic:masterfrom
jsanz:downgrade/chroma-js
Jan 12, 2026
Merged

Align chroma-js with Kibana#1095
jsanz merged 1 commit intoelastic:masterfrom
jsanz:downgrade/chroma-js

Conversation

@jsanz
Copy link
Copy Markdown
Contributor

@jsanz jsanz commented Jan 8, 2026

Our CI is red because the recent change in Kibana elastic/kibana#244556 pinned chroma-js to a version that is older than ours, hence downgrading our library to align with this PR.

@prodsecmachine
Copy link
Copy Markdown

prodsecmachine commented Jan 8, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@jsanz jsanz added dependencies Pull requests that update a dependency file v8.6.4 labels Jan 8, 2026
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

@jsanz jsanz requested a review from nickpeihl January 8, 2026 12:53
@delanni
Copy link
Copy Markdown
Member

delanni commented Jan 8, 2026

@jsanz - even prior to my PR pinning the version, the resolved version within Kibana was 2.4.2 (https://github.com/elastic/kibana/blob/00023f54627f8163bd12d5c0ad0b09c451c0563a/yarn.lock#L17363-L17366)

@jsanz
Copy link
Copy Markdown
Contributor Author

jsanz commented Jan 9, 2026

@jsanz - even prior to my PR pinning the version, the resolved version within Kibana was 2.4.2 (https://github.com/elastic/kibana/blob/00023f54627f8163bd12d5c0ad0b09c451c0563a/yarn.lock#L17363-L17366)

Yeah, our script only relies on the package.json declarations, and indeed your PR improves our situation, thanks!!

Copy link
Copy Markdown
Contributor

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm!

@jsanz jsanz merged commit 4f6306f into elastic:master Jan 12, 2026
6 checks passed
@jsanz jsanz deleted the downgrade/chroma-js branch January 12, 2026 18:30
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 v8.6.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants