-
Notifications
You must be signed in to change notification settings - Fork 147
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 absolute paths #203
Fix absolute paths #203
Conversation
/CC @sindresorhus |
@@ -1,4 +1,7 @@ | |||
'use strict'; | |||
function absolutePath(path) { | |||
return __dirname + '/' + path; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should use path.join()
.
Good points. Do you mind if I just amend the existing commit, or would you prefer a separate one? |
@smcgivern: just amend. |
81586b2
to
4deac5b
Compare
OK, done 👍 |
I know this is already merged, but.. Why are you setting:
When it should be relative to the destination:
|
Honestly, because it was like that before: grunt-contrib-cssmin/tasks/cssmin.js Line 45 in 4af3941
This also isn't merged - @XhmikosR a little help please? 😃 |
This should fix #191 and maybe #196 too.
The
relativeTo
option was removed at https://github.com/gruntjs/grunt-contrib-cssmin/pull/170/files#diff-f0ee9eb300066e1e51fcfbad9d7a78e1L45 and seemingly never added back.