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

Use ReactDOM webpack external #224

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Conversation

mickmister
Copy link
Contributor

@mickmister mickmister commented Sep 21, 2022

Summary

This plugin currently doesn't use react-dom or react-select, so nothing should have broken here. One of these libraries may be used in the future.

Ticket Link

Fixes https://mattermost.atlassian.net/browse/MM-47046

@mickmister mickmister requested a review from larkox as a code owner September 21, 2022 04:32
@codecov-commenter
Copy link

Codecov Report

Base: 45.47% // Head: 45.47% // No change to project coverage 👍

Coverage data is based on head (5bc57ac) compared to base (e09c61d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #224   +/-   ##
=======================================
  Coverage   45.47%   45.47%           
=======================================
  Files           7        7           
  Lines         873      873           
=======================================
  Hits          397      397           
  Misses        451      451           
  Partials       25       25           

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mickmister mickmister merged commit a1f51cd into master Oct 4, 2022
@mickmister mickmister deleted the react-dom_webpack-external branch October 4, 2022 03:33
mickmister added a commit that referenced this pull request Oct 28, 2022
This was referenced Oct 28, 2022
mickmister added a commit that referenced this pull request Nov 3, 2022
* react-dom_webpack-external (#224)

* bump version 2.0.1

* MM-40214: Add App Bar icon (#214)

* go get -u golang.org/x/sys

* Fix CI

* Add App Bar icon

* Fix errors found by linter

* Allow arrow functions in eslint's func-style rule

* fix type mismatch at redux Provider

Co-authored-by: Ngoan Tran <[email protected]>

* go mod tidy

* remove npm i --verbose

* Fix linter errors (#227)

* lint

Co-authored-by: Alejandro García Montoro <[email protected]>
Co-authored-by: Ngoan Tran <[email protected]>
Co-authored-by: Ben Schumacher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants