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

Update all (non-dev) dependencies #195

Closed
wants to merge 2 commits into from
Closed

Update all (non-dev) dependencies #195

wants to merge 2 commits into from

Conversation

tomByrer
Copy link

This combines updating core-js & better buble, & updates all other dependencies (but not devDependencies). buble-jsx-only is a fork that has been battle tested in 11k repos via MDX.

Tests work, lint does not from pre-existing errors..

jest
 PASS  src\utils\test\errorBoundary.test.js
 PASS  src\utils\test\transpile.test.js

Test Suites: 2 passed, 2 total
Tests:       13 passed, 13 total
Snapshots:   0 total
Time:        2.421s
Ran all test suites.

Thanks for your time :)

@tomByrer tomByrer changed the title update all (non-dev) dependencies *DO NOT MERGE* update all (non-dev) dependencies Mar 20, 2020
@tomByrer
Copy link
Author

Looking at the new built file sizes, the updates somehow made the /dist folder grow over 10x.

@tomByrer
Copy link
Author

This version is OK, adds only a few bytes, tests are fine.

@tomByrer tomByrer changed the title *DO NOT MERGE* update all (non-dev) dependencies Update all (non-dev) dependencies Mar 20, 2020
@NateRadebaugh
Copy link

Deploy failed

failed during stage 'building site': Deploy directory 'demo/out' does not exist

@tomByrer
Copy link
Author

Deploy failed

@NateRadebaugh Thanks for looking. Only 2 files I touched was package.json (& nothing related to builds) & src/utils/transpile/transform.js , so I'm not sure how the deployment system got mangled?

@jpdriver jpdriver mentioned this pull request Jun 19, 2021
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.

2 participants