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 grunt dist under Bower by separating docs into separate task #14394

Merged
merged 1 commit into from
Aug 15, 2014

Conversation

cvrebert
Copy link
Collaborator

Fixes #13949.
Also does some refactoring.
CC: @twbs/team

πŸ‘Š
🐦 πŸ”«
✊

@cvrebert cvrebert added this to the v3.2.1 milestone Aug 14, 2014
@cvrebert cvrebert added the grunt label Aug 14, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented Aug 14, 2014

πŸŽ‰ Emoji hype! πŸ‘ on the changes.

@cvrebert
Copy link
Collaborator Author

Confirming that this fixes the issue:

$ bower install bootstrap#gruntfile-bower
bower bootstrap#gruntfile-bower       not-cached git://github.com/twbs/bootstrap.git#gruntfile-bower
bower bootstrap#gruntfile-bower          resolve git://github.com/twbs/bootstrap.git#gruntfile-bower
bower bootstrap#gruntfile-bower         checkout gruntfile-bower
bower bootstrap#gruntfile-bower         resolved git://github.com/twbs/bootstrap.git#06c4002019
bower jquery#>= 1.9.0                     cached git://github.com/jquery/jquery.git#2.1.1
bower jquery#>= 1.9.0                   validate 2.1.1 against git://github.com/jquery/jquery.git#>= 1.9.0
bower bootstrap#gruntfile-bower          install bootstrap#06c4002019
bower jquery#>= 1.9.0                    install jquery#2.1.1

bootstrap#06c4002019 bower_components/bootstrap
└── jquery#2.1.1

jquery#2.1.1 bower_components/jquery
$ cd bower_components/bootstrap
$ npm install
<snip>
$ grunt dist
Running "clean:dist" (clean) task
>> 1 path cleaned.

Running "less:compileCore" (less) task
File dist/css/bootstrap.css.map created.
File dist/css/bootstrap.css created: 134.54 kB β†’ 134.58 kB

Running "less:compileTheme" (less) task
File dist/css/bootstrap-theme.css.map created.
File dist/css/bootstrap-theme.css created: 17.93 kB β†’ 17.98 kB

Running "autoprefixer:core" (autoprefixer) task
File dist/css/bootstrap.css created.
File dist/css/bootstrap.css.map created (source map).

Running "autoprefixer:theme" (autoprefixer) task
File dist/css/bootstrap-theme.css created.
File dist/css/bootstrap-theme.css.map created (source map).

Running "usebanner:files" (usebanner) task
βœ” grunt-banner completed successfully

Running "csscomb:dist" (csscomb) task
>> Using custom config file "less/.csscomb.json"...
>> Sorting file "dist/css/bootstrap-theme.css"...
>> Sorting file "dist/css/bootstrap.css"...

Running "cssmin:minifyCore" (cssmin) task
File dist/css/bootstrap.min.css created: 135.87 kB β†’ 112.48 kB

Running "cssmin:minifyTheme" (cssmin) task
File dist/css/bootstrap-theme.min.css created: 21.45 kB β†’ 18.93 kB

Running "copy:fonts" (copy) task
Copied 4 files

Running "concat:bootstrap" (concat) task
File dist/js/bootstrap.js created.

Running "uglify:core" (uglify) task

Done, without errors.


Execution Time (2014-08-15 17:47:54 UTC)
less:compileCore     1.2s  β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡ 29%
less:compileTheme    96ms  β–‡β–‡β–‡β–‡ 2%
autoprefixer:core   691ms  β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡ 18%
autoprefixer:theme   96ms  β–‡β–‡β–‡β–‡ 2%
csscomb:dist         1.5s  β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡ 37%
uglify:core         385ms  β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡β–‡ 10%
Total 3.9s

$

cvrebert added a commit that referenced this pull request Aug 15, 2014
Fix `grunt dist` under Bower by separating docs into separate task
@cvrebert cvrebert merged commit a9b7ba4 into master Aug 15, 2014
@cvrebert cvrebert deleted the gruntfile-bower branch August 15, 2014 17:52
@cvrebert cvrebert mentioned this pull request Aug 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error trying to build using Bower & Grunt
2 participants