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

Fix webpack overriding && Add an example with local file dependencies #965

Merged
merged 7 commits into from
Apr 27, 2017

Conversation

ndelangen
Copy link
Member

Issue: -can't find it-

What I did

Fixed a few prepublish stuff
Fixed the babelconfig so it loads presets from included 'node_modules'

How to test

  • npm install
  • cd examples/test-cra
  • npm run storybook

This should result in a successful storybook being launched, and no messages in the terminal about presets not being loaded.

for testing a non-linked example based on create-react-app
Copy link
Member

@xavxyz xavxyz left a comment

Choose a reason for hiding this comment

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

🎉 🎉

"version": "0.1.0",
"private": true,
"devDependencies": {
"@kadira/storybook": "file:../../packages/react-storybook",
Copy link
Member

Choose a reason for hiding this comment

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

so cool to have direct dependencies on the file system! 💯

Copy link
Member Author

Choose a reason for hiding this comment

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

This is to be able to do an actual npm install without linking via lerna.
So when everything is installed flat things should work as expected.

},
"dependencies": {
"react": "^15.4.2",
"react-dom": "^15.4.2"
Copy link
Member

Choose a reason for hiding this comment

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

should we be on ^15.5.4 as the latest version published?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah for some reason npm-check-updates is no longer helping me with this 😞

@ndelangen ndelangen merged commit 1e840db into master Apr 27, 2017
@ndelangen ndelangen deleted the webpack-babel-fixes branch April 27, 2017 22:32
@ndelangen ndelangen added the bug label May 27, 2017
@ndelangen ndelangen changed the title FIX webpack overriding && CLEANUP && ADD example Fix webpack overriding && Add an example with local file dependencies May 27, 2017
Copy link

nx-cloud bot commented Mar 7, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit eb2eb55. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants