Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Upgrading emotion from 9 to 11 #216

Merged
merged 8 commits into from
May 14, 2022
Merged

Upgrading emotion from 9 to 11 #216

merged 8 commits into from
May 14, 2022

Conversation

Dezzymei
Copy link
Contributor

@Dezzymei Dezzymei commented Mar 29, 2022

So as to fix #135 with the warnings.

@Dezzymei
Copy link
Contributor Author

It failed only on the build size... I can work on making this smaller I guess.

@pooyaj
Copy link
Contributor

pooyaj commented Apr 7, 2022

Actually I think the size is fine given only about 4kbs

@pooyaj
Copy link
Contributor

pooyaj commented Apr 7, 2022

@Dezzymei would you mind increasing the size limit in your PR so that the build passes, and then I'll merge this in.

@nd4p90x
Copy link
Collaborator

nd4p90x commented Apr 27, 2022

@Dezzymei if you don't mind increasing the yarn size limit, so we will get this merged in. Thanks!

@Dezzymei
Copy link
Contributor Author

Dezzymei commented May 2, 2022

Interestingly running it locally suggests there are no size problems which is strange! I will update it to 55 anyway

@Dezzymei
Copy link
Contributor Author

Dezzymei commented May 2, 2022

Done!

@pooyaj
Copy link
Contributor

pooyaj commented May 2, 2022

@Dezzymei looks like the recent commits removed all the changes. Now it is just the size limit change

@Dezzymei
Copy link
Contributor Author

Dezzymei commented May 3, 2022

Oops - reverted the revert (don't know why I did that!)

Copy link
Contributor

@edsonjab edsonjab left a comment

Choose a reason for hiding this comment

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

The upgrade works, all the components shows correctly

@nd4p90x nd4p90x merged commit 877e4d8 into segmentio:master May 14, 2022
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.

Warning: componentWillMount has been renamed, and is not recommended for use.
4 participants