feat(docs), feat(ci): minor project config

Author: ProGM

.github/workflows/test.yml

@@ -5,7 +5,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [ '12', '14', '16' ]
+        node: [ '14', '16', '18' ]
     name: Node ${{ matrix.node }} sample
     steps:
       - uses: actions/checkout@v2

README.md

@@ -1,3 +1,6 @@
+![Tests](https://github.com/monade/eslint-plugin-angular/actions/workflows/test.yml/badge.svg)
+[![npm version](https://badge.fury.io/js/@monade%2Feslint-plugin-angular.svg)](https://badge.fury.io/js/@monade%2Feslint-plugin-angular)
+
 # @monade/eslint-plugin-angular
 A couple of quality-of-life eslint rules for Angular components.
 
@@ -30,10 +33,13 @@ Add to your .eslint.js:
 
 ## TODO:
 * Document rules
-* When strategy is OnPush, show an error if there's a non-scalar property that is not ReadOnly<>
+* [Rule] When strategy is OnPush, show an error if there's a non-scalar property that is not ReadOnly<>
+* [Rule] Component too long
+* [Rule] Suggest strategy OnPush when the component is in the "components" folder
+* [Rule] ADVANCED: Check if there's a reactivity bug with changeDetection: OnPush
 
-## Debugging
-You can use an AST view tool
+## Contributing
+You can use an AST view tool to debug the structure of some code parts:
 * For Javascript: https://astexplorer.net/
 * For Typescript: https://ts-ast-viewer.com/
 

package.json

@@ -1,17 +1,17 @@
 {
-	"name": "@monade/eslint-plugin-angular",
-	"version": "1.0.0",
-	"description": "A couple of rules for eslint + angular",
-	"main": "src/index.js",
-	"scripts": {
-		"test": "mocha"
-	},
-	"author": "Mònade",
-	"license": "MIT",
-	"devDependencies": {
-		"@typescript-eslint/parser": "^5.35.1",
-		"eslint": "^8.22.0",
-		"mocha": "^10.0.0",
-		"typescript": "^4.7.4"
-	}
+  "name": "@monade/eslint-plugin-angular",
+  "version": "0.1.0",
+  "description": "A couple of quality-of-life rules for eslint + angular",
+  "main": "src/index.js",
+  "scripts": {
+    "test": "mocha"
+  },
+  "author": "Mònade",
+  "license": "MIT",
+  "devDependencies": {
+    "@typescript-eslint/parser": "^5.35.1",
+    "eslint": "^8.22.0",
+    "mocha": "^10.0.0",
+    "typescript": "^4.7.4"
+  }
 }

src/rules/strategy-onpush.js

@@ -29,6 +29,7 @@ module.exports = {
           return;
         }
 
+        /** @type {import('estree').MethodDefinition} */
         const constructor = node.body.body.find(
           (e) => e.kind === "constructor"
         );