This repository has been archived by the owner on May 25, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 310
generator always includes bootstrap.js? #42
Comments
@sindresorhus @passy, should we remove bootstrap js by default? I agree with @stream7 that this is kinda unexpected behaviour. |
Yes from me, but it's @addyosmani or @paulirish that wanted it by default. |
+1 for not shipping the file when disabled |
ping @addyosmani @paulirish |
Yes, It breaks my perfect workflow with yo webapp generator. |
I'm +1 on not shipping the file when disabled as well. |
Okay, I'll remove it for now. |
stick a knife in it. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
16:15 ~/Code/apps/web (yeoman) $ yo webapp
---------´ | Welcome to Yeoman, | ( _´U
_ ) | ladies and gentlemen! |/_A\ '__________________________'
| ~ |
**'.**.'__
´
|° ´ Y
Out of the box I include HTML5 Boilerplate, jQuery and Modernizr.
Would you like to include Twitter Bootstrap for Sass? (Y/n) n
Would you like to include RequireJS (for AMD support)? (Y/n) y
create Gruntfile.js
create package.json
create .gitignore
create .gitattributes
create .bowerrc
create component.json
create .jshintrc
create .editorconfig
create app/favicon.ico
create app/404.html
create app/robots.txt
create app/.htaccess
create app/scripts/vendor/bootstrap.js
create app/styles/main.css
create app/scripts/app.js
create app/index.html
create app/scripts/main.js
create app/scripts/hello.coffee
invoke mocha:app
create test/index.html
create test/lib/chai.js
create test/lib/expect.js
create test/lib/mocha/mocha.css
create test/lib/mocha/mocha.js
create test/spec/test.js
I'm all done. Just run npm install && bower install to install the required dependencies.
As you see in the log it creates app/scripts/vendor/bootstrap.js
Why is that? Since I answered "No" to bootstrap sass I would expect to skip the js too.
The text was updated successfully, but these errors were encountered: