search

miensol / grunt-concat-in-order

Gruntjs plugin for concatenating files with dependency tree topological sort preserved.
MIT License
21 stars 9 forks source link

Question about configuration #4

Closed g-kanoufi closed 10 years ago

g-kanoufi commented 10 years ago

Hello, sorry to bother you with that, it's probably not an issue but more a configuration problem because of my lack of understanding. I tried to add your grunt plugin for the file based addition of .js files. Using the code provided on your readme file, i have an error saying "'path' is not defined.", so here is my question, where and what should be the path defined as.

For now i am using your plugin manually with the default options and it work great, but i really wish i could use it the way it should be.

Thank you.

miensol commented 10 years ago

Hi, can you possibly post the Gruntfile with configuration that isn't working for you? 28 lut 2014 06:55 "g-kanoufi" notifications@github.com napisa³(a):

Hello, sorry to bother you with that, it's probably not an issue but more a configuration problem because of my lack of understanding. I tried to add your grunt plugin for the file based addition of .js files. Using the code provided on your readme file, i have an error saying "'path' is not defined.", so here is my question, where and what should be the path defined as.

For now i am using your plugin manually with the default options and it work great, but i really wish i could use it the way it should be.

Thank you.

Reply to this email directly or view it on GitHubhttps://github.com/miensol/grunt-concat-in-order/issues/4 .

miensol commented 10 years ago

It's just a guess (based on the example configuration) but maybe you need path module like so: var path = require('path'); ?

g-kanoufi commented 10 years ago

With the var path = require(‘path'), it’s now working.

This is really awesome, thank you so much.

Guillaume Kanoufi

On 28 Feb 2014, at 15:36, Piotr Mionskowski notifications@github.com wrote:

It's just a guess (based on the example configuration) but maybe you need path module like so: var path = require('path'); ?

— Reply to this email directly or view it on GitHub.

miensol commented 10 years ago

Glad it worked :)

g-kanoufi commented 10 years ago

Thanks again Piotr!

Guillaume Kanoufi

On 02 Mar 2014, at 17:22, Piotr Mionskowski notifications@github.com wrote:

Glad it worked :)

— Reply to this email directly or view it on GitHub.