Skip to content

use @babel/core #7093

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
Aug 13, 2024
Merged

Conversation

birkskyum
Copy link
Contributor

@birkskyum birkskyum commented Aug 13, 2024

When clean installing (no node_modules) plotly.js, the terminal prompts:

npm warn deprecated [email protected]: In 6.x, the babel package has been deprecated

It's because it's been moved from babel (v6) to @babel/core (v7+)

@archmoj
Copy link
Contributor

archmoj commented Aug 13, 2024

Thanks for the PR.
Please download and update stackgl_modules/index.js from https://app.circleci.com/pipelines/github/plotly/plotly.js/11033/workflows/6e2e50d2-c98f-41da-b16b-400e20e7d155/jobs/244231/artifacts and push

@birkskyum birkskyum force-pushed the Upgrade-babel-to-babel/core branch from d5f1b48 to 0ceaf81 Compare August 13, 2024 14:15
@birkskyum birkskyum force-pushed the Upgrade-babel-to-babel/core branch from 5aa9113 to 431df55 Compare August 13, 2024 15:10
Copy link
Contributor

@archmoj archmoj left a comment

Choose a reason for hiding this comment

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

💃

@archmoj archmoj merged commit 33d5578 into plotly:master Aug 13, 2024
1 check passed
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