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

[NO JIRA]: Add missing babel dep to remove missing warning #187

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

olliecurtis
Copy link
Member

@olliecurtis olliecurtis commented Mar 27, 2024

Adding @babel/plugin-proposal-private-property-in-object as this solves the output warning that consumers get when installing BRS

One of your dependencies, babel-preset-react-app, is importing the
"@babel/plugin-proposal-private-property-in-object" package without
declaring it in its dependencies. This is currently working because
"@babel/plugin-proposal-private-property-in-object" is already in your
node_modules folder for unrelated reasons, but it may break at any time.

babel-preset-react-app is part of the create-react-app project, which
is not maintianed anymore. It is thus unlikely that this bug will
ever be fixed. Add "@babel/plugin-proposal-private-property-in-object" to
your devDependencies to work around this error. This will make this message
go away.

When you install the recommended dependency either in the consumer or this project it presents

npm WARN deprecated @babel/plugin-proposal-private-property-in-object@7.21.11: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-private-property-in-object instead.

So this installs the correctly required package!

@olliecurtis olliecurtis force-pushed the no-jira-update-dev-deps branch from 6a24401 to 7c50fee Compare March 27, 2024 15:57
dominicfraser
dominicfraser previously approved these changes Mar 27, 2024
@xalechez xalechez merged commit 15ac238 into main Apr 4, 2024
4 checks 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.

3 participants