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

Upgrade deps #1261

Merged
merged 4 commits into from
Jul 16, 2021
Merged

Upgrade deps #1261

merged 4 commits into from
Jul 16, 2021

Conversation

mshwery
Copy link
Contributor

@mshwery mshwery commented Jul 15, 2021

Overview
This PR updates all deps within their semver ranges, and uses react-transition-group's nodeRef to abandon findDOMNode (not supported in strict mode). I removed react-scrollbar-size because it's not referenced anywhere.

Screenshots (if applicable)
NA

Documentation

  • Updated Typescript types and/or component PropTypes
  • Added / modified component docs
  • Added / modified Storybook stories

@netlify
Copy link

netlify bot commented Jul 15, 2021

✔️ Deploy Preview for evergreen-storybook ready!

🔨 Explore the source changes: e6d5e2d

🔍 Inspect the deploy log: https://app.netlify.com/sites/evergreen-storybook/deploys/60f0ed86ec5d2d0008b30b88

😎 Browse the preview: https://deploy-preview-1261--evergreen-storybook.netlify.app

Copy link
Contributor

@akleiner2 akleiner2 left a comment

Choose a reason for hiding this comment

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

Need to bump size limit (wondering what deps caused the jump), but otherwise LGTM

@mshwery
Copy link
Contributor Author

mshwery commented Jul 15, 2021

Need to bump size limit (wondering what deps caused the jump), but otherwise LGTM

New Blueprintjs icons from that update.

@mshwery
Copy link
Contributor Author

mshwery commented Jul 16, 2021

Need to bump size limit (wondering what deps caused the jump), but otherwise LGTM

btw @akleiner2 it was the blueprintjs icons – CI builds and then runs size-limit, they just werent included in the git commit.

@mshwery mshwery merged commit a8288d0 into master Jul 16, 2021
@mshwery mshwery deleted the upgrade-deps branch July 16, 2021 02:27
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.

None yet

3 participants