chore(jshint): enforce jshint for tests #7264
Conversation
|
Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.
If you need to make changes to your pull request, you can update the commit with Thanks again for your help! |
|
This loads very slowly in Github since there are a lot of differences. Use this URL in order to ignore whitespace differences: https://github.com/angular/angular.js/pull/7264/files?w=1 (it will load much fatser) |
|
Very nice work sir! |
| @@ -107,6 +107,9 @@ module.exports = function(grunt) { | |||
| options: { | |||
| jshintrc: true, | |||
| }, | |||
| tests: { | |||
| files: { src: 'test/**/*.js' }, | |||
There was a problem hiding this comment.
It is best to use files[...] here so that we keep all file references in a single place, no?
There was a problem hiding this comment.
Oh, I see that we don't do it for the other modules!
|
Do you fancy doing the same for the 1.2.x branch? :-) |
No description provided.