Skip to content

Update various npm packages #276

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

Merged
merged 2 commits into from
Apr 30, 2021
Merged

Conversation

SeanHealy33
Copy link
Contributor

@SeanHealy33 SeanHealy33 commented Apr 27, 2021

Why?

This repo's packages are a little old. This pr aims to update the packages that are used here to be more inline with modern updates of the packages

@johnnynader johnnynader force-pushed the seanhealy/updateVariousPackages branch from eafb3df to f444852 Compare April 29, 2021 11:36
@johnnynader johnnynader force-pushed the seanhealy/updateVariousPackages branch from f444852 to 457e73d Compare April 29, 2021 11:52
@ashorttio ashorttio marked this pull request as ready for review April 29, 2021 14:27
@ashorttio
Copy link

This looks fine. Yarn/NPM are both handling dependency resolution. The upgrade to CircleCI is also executing. What I would say is there is still quite a few deprecated packages. This might need a few more passes.

@johnnynader johnnynader merged commit 48e4dc8 into master Apr 30, 2021
@johnnynader johnnynader deleted the seanhealy/updateVariousPackages branch April 30, 2021 13:02
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