-
Notifications
You must be signed in to change notification settings - Fork 61
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
Support full spectrum of ways to specify files in grunt config #54
Comments
For reference http://gruntjs.com/configuring-tasks#files |
I suspect part of the problem is that grunt-newer parses files out of the config (https://github.com/tschaub/grunt-newer/blob/master/tasks/newer.js#L64) rather than using |
Thanks for the ticket. Keep in mind that The issue above is that the value itself is an array instead of a config object. This condition should be checked when the task config is originally accessed. The code currently only works with config objects. It needs to account for array or string values as well. |
See https://github.com/tschaub/grunt-newer/compare/moar-files-config for a potential fix. Still needs tests. |
Fair enough. I haven't read the code in detail, just figured that grunt must be doing that itself somewhere too.. |
FYI, I've applied the proposed patch and I get when one file is updated:
Without the patch, it would go over all files every time, files updated or not. |
I'd the same issue and I solved it using src instead of files.
|
I confirm @mquintal option: few days ago, I rewrote all my settings for jsbeautifier, jshint, stylud, copy, uncss, and jsdoc with the |
+1 |
+1, same issue. |
+1 |
+1, experiencing it with eslint
eslint: {
target: {
src: ['file1.js']
}
} |
@tschaub, why the suggested fix (https://github.com/tschaub/grunt-newer/compare/moar-files-config) hasn't been merged yet? Do you wait for someone with tested PR? |
I found grunt-newer worked if I did
but not for
It seems to only understand one syntax for
files
but grunt supports multiple ways of specifying.Refs #39
The text was updated successfully, but these errors were encountered: