-
-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
Installing Storybook 2.35.3 results in babel deprecation warning #756
Comments
Thanks for the report! |
I've worked on moving devDependencies to the root today, mostly babel. So Will need to check if this is still something that happens. I'm not getting any error-messages like this. |
If someone could check if this still happens and if so when, that'd be a great help! |
I'm happy to help out, but I don't see any releases after |
Hey thanks @kaiyoma ! We recently moved to a monorepo structure managed by lerna.
Thanks so much! Let me know if you need help. |
I can't repro the issue by cloning the repo and installing. To repro the issue, you have to install storybook from inside another project. You can do so with this barebones config: package.json:
.npmrc:
When you |
Right so that will install an older version, I'd love to know if what's on master still exhibits this behavior.. |
I wonder if that requires some |
See #756 (comment) We should actually ensure this works consistently for people before making this change.
Looks like we're using babel-preset-latest somewhere we should be using babel-preset-env |
@kaiyoma We now have some steps described here: https://github.com/storybooks/storybook/blob/master/CONTRIBUTING.md#testing-against-master I hope this helps you reproduce the issue. |
I'm getting an error when running
|
In my team's project, we need an entry in our
Maybe you need the same thing in your source tree? |
Looking at your log, It's clear the package it's failing to install is a dependency of 'jest-enzyme'.
I have no idea why this would be necessary, your node & npm versions seem fine to me. |
Adding In the interest of full helpfulness, I noticed a few (probably benign) warnings during the installation:
During
However, this time, adding a line to
And here's what happens when I try
That error repeats about 4 times and I also see this:
Any ideas? (BTW, how come you guys require the |
Hey @kaiyoma, I'm at a loss.. The I'm aware of this one:
But these are unknown to me:
We're migrating away from the @kadira organisation to a @storybook organisation on npm. But this is still in progress. |
We have an internal NPM registry here at my organization. Would that explain things? |
Yeah that would have to be it! Maybe you you can ask for support there. At my current client (unrelated to storybook), we're using artifactory as a custom registry, and it's able to pass though to the regular registry (including organisations) just fine. |
Our internal repository does the same thing actually. I'm wondering if the |
I'm running into issues on my personal laptop as well. I believe I made it past the name resolution part and I'm now seeing an error relating to a missing directory:
I looked at the directory structure manually and I can verify that there is definitely no
|
This will be a known issue in the |
It seems that |
When I do a clean CRA install and then add storybook via cli, I do not get this message anymore. |
See #756 (comment) We should actually ensure this works consistently for people before making this change.
npm ERR! code E404 |
Did you copy that output exactly? Because you have a typo here:
That should be |
thanks for that i will rerun it |
Here are the relevant portions of my
package.json
:If I blow away
node_modules
and perform a cleannpm install
, I get this warning:I'm not installing
babel-preset-latest
orbabel-preset-env
directly; they're both coming from Storybook. Please let me know if you need any more info.The text was updated successfully, but these errors were encountered: