Skip to content
This repository has been archived by the owner on Mar 26, 2018. It is now read-only.

feat(build): add autoprefixer support #317

Closed
wants to merge 1 commit into from

Conversation

passy
Copy link
Member

@passy passy commented Aug 16, 2013

This is literally the first version I wrote for this, but I couldn't find any bugs. Which can only mean that my approach to testing sucks.

@addyosmani You're probably still familiar with this from your update to generator-webapp. you mind looking over this?

Add support for SASS and vanilla CSS to get automatic vendor prefixes base on
the browsers selected in the Gruntfile.
@@ -16,6 +16,7 @@
"grunt-contrib-htmlmin": "~0.1.3",
"grunt-contrib-imagemin": "~0.1.4",
"grunt-contrib-watch": "~0.4.0",
"grunt-autoprefixer": "~0.2.20130806",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i know it's correct, but it looks so ugly. rather: ~0.2.2

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree. Was actually looking at that very line in the PR.
On Aug 16, 2013 6:20 PM, "Sindre Sorhus" notifications@github.com wrote:

In templates/common/_package.json:

@@ -16,6 +16,7 @@
"grunt-contrib-htmlmin": "~0.1.3",
"grunt-contrib-imagemin": "~0.1.4",
"grunt-contrib-watch": "~0.4.0",

  • "grunt-autoprefixer": "~0.2.20130806",

i know it's correct, but it looks so ugly. rather: ~0.2.2


Reply to this email directly or view it on GitHubhttps://github.com//pull/317/files#r5817661
.

@sindresorhus
Copy link
Member

Can you fix the merge conflict?

@addyosmani ping

@passy passy closed this in c4dfd61 Aug 17, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants