search

opensearch-project / dashboards-flow-framework

A UI designer for constructing AI applications with OpenSearch
Apache License 2.0
9 stars 6 forks source link

Remove unnecessary `js-yaml` package #240

Closed ohltyler closed 1 month ago

ohltyler commented 1 month ago

Description

Core OSD already has the js-yaml dependency managed. We remove the duplicate pulled into this plugin which can lead to build issues due to version mismatches. Confirmed everything works and the js-yaml uses in this codebase resolve correctly.

Issues Resolved

N/A

Check List

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

ohltyler commented 1 month ago

test failure is unrelated, I've checked on much older branches, all fail the same when using the latest branch of OSD. Started failing on #231 . Will ignore for this PR. Still investigating