-
Notifications
You must be signed in to change notification settings - Fork 12
fix(build): pin react-bootstrap; upgrade to react v15 #3
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
base: master
Are you sure you want to change the base?
Conversation
This looks great! Honestly, the only thing I'd change is the React Bootstrap versioning to |
hi @pxpeterxu thanks for the reply! I've closed the PR before because I used the branch I'll update the PR to merge only the part before I replaced the |
Thanks for this fix, @gabrielrtakeda! Looking forward to the merge... |
@gabrielrtakeda is there any plan to merge this PR to the main repo and release a new version? It's currently unusable when referenced from NPM. |
related: #2
before
Full log
after