search

firstandthird / load-grunt-config

Grunt plugin that lets you break up your Gruntfile config by task
firstandthird.github.io/load-grunt-config/
MIT License
374 stars 64 forks source link

Use array for default value of configPath #141

Closed schmengler closed 8 years ago

schmengler commented 9 years ago

Currently if I configure "configPath" as an array, merging it with the default values results in an array like this:

[
  '/my/path1',
  '/my/path2';
  'a',
  'r',
  '/',
  'w'
// ...
]

Because the string is treated as an array of characters, when used as array.

(I'm running node.js 5.0.0 if that matters)

SolomoN-ua commented 9 years ago

@schmengler thx for PR, but can you explain why you need to configure configPath as an array?

schmengler commented 9 years ago

I want to read configuration from multiple directories. According to the README this should be possible:

configPath and overridePath accept single string as well as array of strings. It means that you can compose config using multiple folders. For example:

(and it already was possible before I updated everything recently)

SolomoN-ua commented 9 years ago

Please give me some time, I will accept this PR after finishing work with Travis. Thanks :)

SolomoN-ua commented 8 years ago

@schmengler thanks for your PR.