Skip to content

Upgrade @babel/plugin-* and @babel/preset-* packages to latest#6662

Merged
cee-chen merged 5 commits intoelastic:mainfrom
cee-chen:babel-upgrade-1
Mar 30, 2023
Merged

Upgrade @babel/plugin-* and @babel/preset-* packages to latest#6662
cee-chen merged 5 commits intoelastic:mainfrom
cee-chen:babel-upgrade-1

Conversation

@cee-chen
Copy link
Copy Markdown
Contributor

@cee-chen cee-chen commented Mar 29, 2023

Summary

This PR is part of an ongoing effort to upgrade our various babel dependencies to the latest (#6652) in order for Storybook to work as expected.

I'm opening PRs in very very atomic increments in order to 1. get frequent help from other devs testing, so I don't run into a "it works on my machine" issue, and 2. catch any issues that may arise in CI.

QA

  • CI passes

The following scripts work as expected locally:

  • yarn start
  • yarn lint
  • yarn test-unit
  • yarn test-cypress-dev
  • yarn test-unit accordion --coverage && yarn test-cypress && yarn combine-test-coverage
  • yarn yo-changelog
  • yarn build-pack
  • npm run release (run up to the 2FA prompt and stop)
  • yarn sync-docs

@cee-chen cee-chen added skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) tech debt labels Mar 29, 2023
@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_6662/

@cee-chen cee-chen changed the title Upgrade @babel/plugin-* packages to latest Upgrade @babel/plugin-* and @babel/preset-* packages to latest Mar 29, 2023
@cee-chen
Copy link
Copy Markdown
Contributor Author

@breehall or @daveyholler - as my M1 guinea pigs, can either of you (whoever has spare cycles soonest) QA this PR for me:

  1. gh pr checkout 6662
  2. yarn
  3. yarn start

and confirm the above yarn start works successfully / does not throw errors? If no errors, please go ahead and approve this PR. Thank you!

@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_6662/

Copy link
Copy Markdown
Contributor

@breehall breehall left a comment

Choose a reason for hiding this comment

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

This builds and starts for me with no error!

@cee-chen cee-chen enabled auto-merge (squash) March 30, 2023 17:46
@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_6662/

@cee-chen cee-chen merged commit c276fd4 into elastic:main Mar 30, 2023
@cee-chen cee-chen deleted the babel-upgrade-1 branch March 30, 2023 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants