Skip to content

Chore: update grunt deps #1168

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

Merged
merged 1 commit into from
Aug 14, 2015

Conversation

kingcody
Copy link
Member

Changes:

@Awk34
Copy link
Member

Awk34 commented Aug 14, 2015

oh jeez, why not have these all as one commit?

@kingcody
Copy link
Member Author

I was going to, but I didn't know if we wanted to record it in the changelog.md

@kingcody
Copy link
Member Author

@Awk34 it's your call, I don't really care; can just as easily squash it.

@Awk34
Copy link
Member

Awk34 commented Aug 14, 2015

Yeah, squash em.

@kingcody kingcody force-pushed the chore/update-grunt-deps branch from 2b79a87 to db35124 Compare August 14, 2015 15:51
@kingcody
Copy link
Member Author

done()

@Awk34
Copy link
Member

Awk34 commented Aug 14, 2015

done()

Hah, what a nerd!

Do you think you could just make some notes (here, or body of commit message, etc) about the breaking changes/etc from some of the updated packages?

@kingcody
Copy link
Member Author

There shouldn't be any BC's. Also, not sure how all these other commits got here. I'll rebase...

@kingcody kingcody force-pushed the chore/update-grunt-deps branch from db35124 to 940e08a Compare August 14, 2015 22:54
@Awk34
Copy link
Member

Awk34 commented Aug 14, 2015

Oh, I meant the changes to the Gruntfile.js

Changes:
* `grunt-contrib-watch` now uses `spawn: false` instead of `nospawn: true`
@see https://github.com/gruntjs/grunt-contrib-watch#optionsspawn
* `grunt-contrib-imagemin` now minifies svg files, remove `grunt-svgmin`
@see https://github.com/gruntjs/grunt-contrib-imagemin#imagemin-task
@kingcody kingcody force-pushed the chore/update-grunt-deps branch from 940e08a to aa6047e Compare August 14, 2015 23:13
@kingcody
Copy link
Member Author

There we go.

@Awk34
Copy link
Member

Awk34 commented Aug 14, 2015

thanks

Awk34 added a commit that referenced this pull request Aug 14, 2015
@Awk34 Awk34 merged commit b74e5fc into angular-fullstack:canary Aug 14, 2015
@kingcody kingcody deleted the chore/update-grunt-deps branch August 15, 2015 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants