-
Notifications
You must be signed in to change notification settings - Fork 51
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
Does not work with react-bootstrap #59
Comments
That's because the bower version of react-bootstrap isn't CommonJS compatible, it's made for using in a browser. Per their bower.json, the main file is react-bootstrap.js, a pre-built AMD/browser global file that's not meant to be used inside node or browserify or anything else that uses CommonJS. Note that even if you included So either convince react-bootstrap to use UMD instead of AMD in their pre-built file, or use a non-bower version of react-bootstrap (possibly the npm version?), or try to figure out the exact right deamdify+debowerify+shim combination to trick the bower version into playing nice :) |
Can we at least fail intelligibly? Is it possible in this situation? |
It has nothing to do with debowerify, and everything to do with the fact that calling |
Simple test with react-bootstrap
https://github.com/EvanCarroll/debowerify-fail
react-bootstrap/index.js
has a series of requiresWhile the file,
./Acordion.js
has itself a series of requires.These don't seem to be resolvable with
react-bootstrap
.The text was updated successfully, but these errors were encountered: