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

Avoid unhandled error when used with aliases #61

Merged
merged 1 commit into from
Oct 21, 2014
Merged

Avoid unhandled error when used with aliases #61

merged 1 commit into from
Oct 21, 2014

Conversation

tschaub
Copy link
Owner

@tschaub tschaub commented Oct 21, 2014

Grunt doesn't provide an easy way to access the list of tasks associated with an alias. Instead of an unhandled error, the task should fail.

See #59.

tschaub added a commit that referenced this pull request Oct 21, 2014
Avoid unhandled error when used with aliases.
@tschaub tschaub merged commit 5415f53 into master Oct 21, 2014
@tschaub tschaub deleted the alias branch October 21, 2014 23:12
@tschaub tschaub mentioned this pull request Oct 21, 2014
@FagnerMartinsBrack
Copy link

I opened gruntjs/grunt#1238 in case grunt is willing to add this feature.

@paulsouche
Copy link

hi

Same issue here. Is there a way to launch the two tasks within a watch ?

    watch: {
      options : {
        interrupt: true
      },
      js: {
        files: [...],
        tasks: ['newer:jscs', 'newer:jshint']
      }
    }

Does not work for me

Thanks

@FagnerMartinsBrack
Copy link

It doesn't seems to be related to grunt-newer plugin, it looks like something to be looked up in grunt-contrib-watch docs.

@paulsouche
Copy link

Yeah just asking...

    watch: {
      options : {
        interrupt: true
      },
      jscs: {
        files: [...],
        tasks: ['newer:jscs']
      },
      jshint: {
        files: ['Same files'],
        tasks: ['newer:jshint']
      }
    }

Works but is a little bit verbose

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.

3 participants