chore: update husky to 8.0.3 to avoid use execa #160
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Lin Wang <wonglam@amazon.com>
ruanyl
reviewed
Apr 6, 2023
There was a problem hiding this comment.
@wanglam I remember you mentioned before that if the package version different from the version of other plugins, the build will fail, that's why we use an old version of husky, is that still the case?
There was a problem hiding this comment.
Yes. The details is in this PR. The yarn osd bootstrap will be failed. We will clone many plugins in the development phase. For plugin build phase it always care about one plugin, i am not sure if the dependency install will be failed. I also find some different husky version in dashboards-observability and dashboards-query-workbench.
There was a problem hiding this comment.
I see, if only one plugin is build at one time, I think that will not be an issue.
There was a problem hiding this comment.
Will check with @peterzhuamazon about the different dev dependencies.
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #160 +/- ##
=======================================
Coverage 78.32% 78.32%
=======================================
Files 21 21
Lines 383 383
Branches 74 74
=======================================
Hits 300 300
Misses 60 60
Partials 23 23 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
ruanyl
approved these changes
Apr 6, 2023
wanglam
changed the title
wanglam
changed the title
|
Change to draft cause not a critical issue. |
opensearch-trigger-bot bot
pushed a commit
that referenced
this pull request
May 23, 2023
Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 6a3faee)
wanglam
added a commit
that referenced
this pull request
May 23, 2023
wanglam
added a commit
that referenced
this pull request
Jun 15, 2023
* Increment version to 2.8.0.0 (#169) Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> * chore: update husky to 8.0.3 to avoid use execa (#160) (#189) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 6a3faee) Co-authored-by: Lin Wang <wonglam@amazon.com> * doc: add release notes for 2.8.0 (#190) (#191) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit f2b701b) Co-authored-by: Lin Wang <wonglam@amazon.com> * feat: remove experiment warning banner (#194) (#196) * feat: remove experiment warning banner Signed-off-by: Lin Wang <wonglam@amazon.com> * doc: update release notes for remove experiment banner Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 0ef4753) Co-authored-by: Lin Wang <wonglam@amazon.com> * feat: enable ml-commons-dashboards plugin by default (#200) (#202) * feat: enable ml-commons-dashboards plugin by default Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: update release notes for enable plugin by default Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit a1cb7de) Co-authored-by: Lin Wang <wonglam@amazon.com> * Revert remove experiment banner and enable by default (#203) (#205) * Revert "feat: enable ml-commons-dashboards plugin by default (#200)" This reverts commit a1cb7de. Signed-off-by: Lin Wang <wonglam@amazon.com> * Revert "feat: remove experiment warning banner (#194)" This reverts commit 0ef4753. Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 31df7d4) Co-authored-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Signed-off-by: Lin Wang <wonglam@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com>
wanglam
added a commit
that referenced
this pull request
Nov 16, 2023
* Increment version to 2.8.0.0 (#169) Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> * chore: update husky to 8.0.3 to avoid use execa (#160) (#189) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 6a3faee) Co-authored-by: Lin Wang <wonglam@amazon.com> * doc: add release notes for 2.8.0 (#190) (#191) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit f2b701b) Co-authored-by: Lin Wang <wonglam@amazon.com> * feat: remove experiment warning banner (#194) (#195) * feat: remove experiment warning banner Signed-off-by: Lin Wang <wonglam@amazon.com> * doc: update release notes for remove experiment banner Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 0ef4753) Co-authored-by: Lin Wang <wonglam@amazon.com> * feat: enable ml-commons-dashboards plugin by default (#200) (#201) * feat: enable ml-commons-dashboards plugin by default Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: update release notes for enable plugin by default Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit a1cb7de) Co-authored-by: Lin Wang <wonglam@amazon.com> * Revert remove experiment banner and enable by default (#203) (#204) * Revert "feat: enable ml-commons-dashboards plugin by default (#200)" This reverts commit a1cb7de. Signed-off-by: Lin Wang <wonglam@amazon.com> * Revert "feat: remove experiment warning banner (#194)" This reverts commit 0ef4753. Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 31df7d4) Co-authored-by: Lin Wang <wonglam@amazon.com> * remove background for dark mode to fix #206 (#209) (#212) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 2bc3f38) Co-authored-by: Lin Wang <wonglam@amazon.com> * Fix ml document link address (#210) (#213) * update to correct machine learning document address to fix #208 Signed-off-by: Lin Wang <wonglam@amazon.com> * fix: update link in deployment table empty screen Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 7a33d01) Co-authored-by: Lin Wang <wonglam@amazon.com> * feat: open external link in blank page (#211) (#214) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit e2825c4) Co-authored-by: Lin Wang <wonglam@amazon.com> * Increment version to 2.9.0.0 (#222) Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> * feat: exclude remote model for admin UI (#225) (#228) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit ba01d34) Co-authored-by: Lin Wang <wonglam@amazon.com> * Revert "Revert remove experiment banner and enable by default (#203)" (#229) (#231) * Revert "Revert remove experiment banner and enable by default (#203)" This reverts commit 31df7d4. Signed-off-by: Lin Wang <wonglam@amazon.com> * doc: remove release update for 2.8.0 Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 2caf020) Co-authored-by: Lin Wang <wonglam@amazon.com> * doc: add release notes for 2.9.0 (#232) (#233) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 02e80a9) Co-authored-by: Lin Wang <wonglam@amazon.com> * Add source field to distinguish local and external model (#239) (#240) * Revert "feat: exclude remote model for admin UI (#225)" This reverts commit ba01d34. Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: add source field to distinguish local and external model Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: add miss display words Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 63c7a5a) Co-authored-by: Lin Wang <wonglam@amazon.com> * Fix no model show up when search a model (#238) (#242) * fix: reset to page 1 after name or state search Signed-off-by: Lin Wang <wonglam@amazon.com> * refactor: remove unnecessary act call Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 24aa9df) Co-authored-by: Lin Wang <wonglam@amazon.com> * Feat mitigate to oui style variables (#227) (#246) * feat: remove customized font size and font weight Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: update color to $ouiLinkColor Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: remove white background for refresh interval Signed-off-by: Lin Wang <wonglam@amazon.com> * remove ace theme Signed-off-by: Lin Wang <wonglam@amazon.com> * refactor: update <b> to <strong> Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit 3aa8eaf) Co-authored-by: Lin Wang <wonglam@amazon.com> * feat: support external models in deployed model list (#248) (#251) * feat: support external models in deployed model list Signed-off-by: Lin Wang <wonglam@amazon.com> * refactor: update name to required Signed-off-by: Lin Wang <wonglam@amazon.com> * fix: connector index not found Signed-off-by: Lin Wang <wonglam@amazon.com> * refactor: update options filter with normal string[] value Signed-off-by: Lin Wang <wonglam@amazon.com> * fix: hits not defined Signed-off-by: Lin Wang <wonglam@amazon.com> * fix: update wording Signed-off-by: Lin Wang <wonglam@amazon.com> * fix: connector id not exists in all connectors Signed-off-by: Lin Wang <wonglam@amazon.com> * fix: show models when failed to load all external connectors Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: update deployed models title to models Signed-off-by: Lin Wang <wonglam@amazon.com> * refactor: remove unused code in model connector filter Signed-off-by: Lin Wang <wonglam@amazon.com> * feat: address PR comments Signed-off-by: Lin Wang <wonglam@amazon.com> --------- Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit cc3810b) Co-authored-by: Lin Wang <wonglam@amazon.com> * update preview panel for external models (#252) (#253) * feat: update preview panel for external models Signed-off-by: tygao <tygao@amazon.com> * feat: update preview panel for external models Signed-off-by: tygao <tygao@amazon.com> --------- Signed-off-by: tygao <tygao@amazon.com> (cherry picked from commit 9bbd25f) Co-authored-by: raintygao <tygao@amazon.com> * Increment version to 2.10.0.0 (#237) Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> * feat: update ui component and use fallback (#254) (#256) Signed-off-by: tygao <tygao@amazon.com> * feat: replace dash with em dash (#255) (#257) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit b817d98) Co-authored-by: Lin Wang <wonglam@amazon.com> * doc: add release notes for 2.10 (#258) (#259) * doc: add release notes for 2.10 Signed-off-by: tygao <tygao@amazon.com> * doc: append prs Signed-off-by: tygao <tygao@amazon.com> * doc: separate prs Signed-off-by: tygao <tygao@amazon.com> --------- Signed-off-by: tygao <tygao@amazon.com> (cherry picked from commit db0003d) Co-authored-by: tygao <tygao@amazon.com> * Increment version to 2.11.0.0 (#265) Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> * doc: add release notes for 2.11.0.0 (#276) (#277) Signed-off-by: Lin Wang <wonglam@amazon.com> (cherry picked from commit cc2d58e) Co-authored-by: Lin Wang <wonglam@amazon.com> * Onboard jenkins build docker image to ml-commons-dashboards github ci checks 2.x (#281) Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Increment version to 2.12.0.0 (#279) Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> --------- Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Signed-off-by: tygao <tygao@amazon.com> Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> Signed-off-by: Lin Wang <wonglam@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: raintygao <tygao@amazon.com> Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The
execawill cause GMS-2020-2 advisor.The old
huskydepends on it, update thehuskyto 8.0.3 , then we won't useexecaanymore.After this PR merged, we need to re-run
yarnto upgrade thehuskyversion.Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
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.