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

Devel fix some problems #3

Closed
wants to merge 3 commits into from
Closed

Devel fix some problems #3

wants to merge 3 commits into from

Conversation

ahomu
Copy link

@ahomu ahomu commented Oct 20, 2012

Specified options.paths then @import nib failed.
And options.urlfunc is ignored.

The version of there are fixed this problems. :)

@tkellen
Copy link
Member

tkellen commented Oct 20, 2012

Thanks Ayumu! Are you using this branch with Grunt .4?

var stylus = require('stylus');
var s = stylus(srcCode);

grunt.util._.each(options, function(value, key) {
Copy link
Member

Choose a reason for hiding this comment

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

Just use native method forEach

Copy link
Author

Choose a reason for hiding this comment

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

That means like Object.keys(options).forEach(function(key) { var value = options[key]; .... ?

@ahomu
Copy link
Author

ahomu commented Oct 21, 2012

Yes, I'm using grunt 0.4.0a and it seems working correctly.

@tkellen
Copy link
Member

tkellen commented Dec 6, 2012

Ack! This should have been merged ages ago Ayumu, I'm very sorry. This has been fixed in another PR, so I am closing it. Grunt 0.4 will be released in the next week or so and you'll be able to get these changes from npm.

@tkellen tkellen closed this Dec 6, 2012
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.

None yet

3 participants