Merge pull request #457 from Gillespie59/development

Development

Author: EmmanuelDemey

package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-angular",
-  "version": "1.6.3",
+  "version": "1.6.4",
   "description": "ESLint rules for AngularJS projects",
   "main": "index.js",
   "scripts": {

rules/di.js

@@ -107,7 +107,7 @@ module.exports = {
                                 return;
                             }
                         }
-                    } else {
+                    } else if (fn.params.length > 0) {
                         report(fn);
                     }
                 } else if (fn.params && fn.params.length !== 0) {

test/di.js

@@ -38,6 +38,9 @@ angularObjectList.forEach(function(object) {
     }, {
         code: 'angular.module("myModule").' + object + '(myFunction);function MyFunction() {}',
         options: ['function']
+    }, {
+        code: 'angular.module("myModule").' + object + '(myFunction);function myFunction() {}',
+        options: ['$inject']
     }, {
         code: 'angular.module("myModule").' + object + '(myFunction);myFunction.$inject=[];function myFunction() {}',
         options: ['$inject']