Skip to content

fix: Add react-dom, redux and react-redux to remote component dependencies#1127

Merged
mofojed merged 1 commit intodeephaven:mainfrom
alexbassy:patch-1
Mar 3, 2023
Merged

fix: Add react-dom, redux and react-redux to remote component dependencies#1127
mofojed merged 1 commit intodeephaven:mainfrom
alexbassy:patch-1

Conversation

@alexbassy
Copy link
Copy Markdown
Contributor

@alexbassy alexbassy commented Mar 3, 2023

Allows plugins to connect to the store and to not have to bundle react-dom

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 3, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@alexbassy
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@dsmmcken dsmmcken requested a review from mofojed March 3, 2023 15:08
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 3, 2023

Codecov Report

Merging #1127 (46fc490) into main (4c7710e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1127   +/-   ##
=======================================
  Coverage   43.38%   43.38%           
=======================================
  Files         435      435           
  Lines       32691    32691           
  Branches     8242     8242           
=======================================
  Hits        14184    14184           
  Misses      18458    18458           
  Partials       49       49           
Flag Coverage Δ
unit 43.38% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ackages/code-studio/src/remote-component.config.js 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Member

@mofojed mofojed left a comment

Choose a reason for hiding this comment

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

Great work, thanks for the contribution!

@mofojed mofojed merged commit d6c8a98 into deephaven:main Mar 3, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 3, 2023
@alexbassy alexbassy deleted the patch-1 branch March 7, 2023 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants