search

jwvdiermen / grunt-include-source

Include lists of files into your source files automatically.
MIT License
68 stars 31 forks source link

Feature: Ability to add files by commas. #9

Closed dixso closed 10 years ago

jwvdiermen commented 10 years ago

You can already do this by doing: <!-- include: "type": "js", "files": [ "scripts/libs/**/*.js", "scripts/app/**/*.js" ] --> I feel simply splitting on a comma in the glob pattern could lead to unexpected results, like having a file name with a comma in it in your Glob pattern.

Unless you have a very good reason to do it, I think the array syntax is a better solution since you can completely rely on the JSON parsing. I'll leave the issue open for now, awaiting your response.

dixso commented 10 years ago

Hi, I think you're right. Thanks for your response.