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

Dependency upgrades #156

Merged
merged 17 commits into from
Aug 7, 2019
Merged

Dependency upgrades #156

merged 17 commits into from
Aug 7, 2019

Conversation

MatthewHerbst
Copy link
Owner

@MatthewHerbst MatthewHerbst commented Aug 6, 2019

Upgrade our dependencies to their latest versions. Steps:

  1. Manually upgraded all listed dependencies
  2. Ran npm audit fix to correct audit issues
  3. Ran npm dedupe to reduce dependency bloat

Also fixed a small bug in the local example.

Build before changes:

Hash: 2627e4d377305a147709
Version: webpack 4.29.6
Time: 1373ms
Built at: 08/06/2019 4:36:20 PM
     Asset       Size  Chunks             Chunk Names
index.d.ts  865 bytes          [emitted]
  index.js   10.9 KiB       0  [emitted]  main
Entrypoint main = index.js
[0] ./src/index.tsx 7.74 KiB {0} [built]

Build after changes:

Hash: db0099965a1b3a7069de
Version: webpack 4.39.1
Time: 1656ms
Built at: 08/06/2019 4:35:13 PM
     Asset      Size  Chunks             Chunk Names
index.d.ts  1.02 KiB          [emitted]
  index.js  10.9 KiB       0  [emitted]  main
Entrypoint main = index.js
[0] ./src/index.tsx 7.74 KiB {0} [built]

This has been giving a 404 for a long time. Removing it does not
impact the example.
@MatthewHerbst MatthewHerbst merged commit 878c877 into MatthewHerbst:master Aug 7, 2019
@MatthewHerbst MatthewHerbst deleted the dependency-upgrades branch August 7, 2019 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant