feat: add eslint

Author: splincode

.editorconfig

@@ -1,13 +1,11 @@
-# Editor configuration, see https://editorconfig.org
+# Editor configuration, see http://editorconfig.org
 root = true
 
 [*]
 charset = utf-8
+indent_size = 4
+end_of_line = lf
 indent_style = space
-indent_size = 2
+max_line_length = 120
 insert_final_newline = true
 trim_trailing_whitespace = true
-
-[*.md]
-max_line_length = off
-trim_trailing_whitespace = false

.eslintrc.json

@@ -0,0 +1,3 @@
+{
+    "extends": "@angular-ru/eslint-config"
+}

.travis.yml

@@ -6,6 +6,7 @@ node_js:
 install:
   - npm install
 script:
+  - npm run lint
   - npm test
   - npm run build
 after_success:

helpers/console-fake.ts

@@ -1,3 +1,4 @@
+/* eslint-disable */
 import { ObjectKeyMap } from '../projects/logger/src/lib/interfaces/logger.internal';
 
 export enum TestLoggerLineType {

helpers/custom-colors.enum.ts

@@ -1,3 +1,4 @@
+// eslint-disable-next-line @typescript-eslint/naming-convention,no-restricted-syntax
 export enum CUSTOM_COLORS {
     TRACE = 'BlueViolet',
     DEBUG = 'CornflowerBlue',
@@ -6,6 +7,7 @@ export enum CUSTOM_COLORS {
     ERROR = 'Crimson'
 }
 
+// eslint-disable-next-line no-restricted-syntax,@typescript-eslint/naming-convention
 export enum CUSTOM_LABELS {
     TRACE = 'trace:',
     DEBUG = 'debug:',

helpers/test.component.ts

@@ -1,17 +1,18 @@
 import { Component, OnInit } from '@angular/core';
-import { LoggerService } from '../projects/logger/src/lib/logger.service';
-import { Logger } from '../projects/logger/src/lib/decorators/logger.decorator';
+
 import { DebugLog } from '../projects/logger/src/lib/decorators/debug.decorator';
-import { TraceLog } from '../projects/logger/src/lib/decorators/trace.decorator';
-import { InfoLog } from '../projects/logger/src/lib/decorators/info.decorator';
 import { ErrorLog } from '../projects/logger/src/lib/decorators/error.decorator';
-import { WarnLog } from '../projects/logger/src/lib/decorators/warn.decorator';
-import { Log } from '../projects/logger/src/lib/decorators/log.decorator';
-import { Group } from '../projects/logger/src/lib/decorators/groups/group.decorator';
 import { GroupCollapsed } from '../projects/logger/src/lib/decorators/groups/group-collapsed.decorator';
+import { Group } from '../projects/logger/src/lib/decorators/groups/group.decorator';
+import { InfoLog } from '../projects/logger/src/lib/decorators/info.decorator';
+import { Log } from '../projects/logger/src/lib/decorators/log.decorator';
+import { Logger } from '../projects/logger/src/lib/decorators/logger.decorator';
 import { TimerLog } from '../projects/logger/src/lib/decorators/timer.decorator';
+import { TraceLog } from '../projects/logger/src/lib/decorators/trace.decorator';
+import { WarnLog } from '../projects/logger/src/lib/decorators/warn.decorator';
 import { LogFn, LoggerLevel, TimerInfo } from '../projects/logger/src/lib/interfaces/logger.external';
 import { Fn } from '../projects/logger/src/lib/interfaces/logger.internal';
+import { LoggerService } from '../projects/logger/src/lib/logger.service';
 
 interface HttpDebugInterface {
     method: string;
@@ -38,6 +39,11 @@ export class MyTestComponent implements OnInit {
     public doneHeavy: boolean = false;
     public name: string = 'MockLoggerComponent';
 
+    public static getUrlInfo({ method, url, queryParams }: Partial<HttpDebugInterface>): string {
+        const params: string = queryParams ? `?${queryParams}` : '';
+        return `[${method}] - ${url}${params}`;
+    }
+
     @Group('Test group')
     public print(val: string): string {
         this.logger.log(val);
@@ -73,23 +79,18 @@ export class MyTestComponent implements OnInit {
         this.count++;
     }
 
-    @Group((name: string) => `Test group with ${name}`)
+    @Group((name: string): string => `Test group with ${name}`)
     public method(name: string): string {
         this.logger.log('group is worked');
         return name;
     }
 
-    @Group((options: Partial<HttpDebugInterface>) => MyTestComponent.getUrlInfo(options))
+    @Group((options: Partial<HttpDebugInterface>): string => MyTestComponent.getUrlInfo(options))
     public hello(name: string): string {
         this.logger.log('group is worked');
         return name;
     }
 
-    public static getUrlInfo({ method, url, queryParams }: Partial<HttpDebugInterface>): string {
-        const params: string = queryParams ? `?${queryParams}` : '';
-        return `[${method}] - ${url}${params}`;
-    }
-
     @TimerLog('mock:ngOnInit')
     public ngOnInit(): void {
         this.hook = 'ngOnInit';
@@ -112,6 +113,7 @@ export class MyTestComponent implements OnInit {
     }
 
     private extracted(seconds: number, done: Fn): void {
+        // eslint-disable-next-line @typescript-eslint/no-magic-numbers
         const e: number = new Date().getTime() + seconds * 1000;
         while (new Date().getTime() <= e) {
             this.doneHeavy = true;

package.json

@@ -10,7 +10,7 @@
         "build:app": "ng build",
         "build.meta": "echo 'prebuild'",
         "test": "jest --config ./jest.app.config.js --coverage",
-        "lint": "ng lint",
+        "lint": "eslint --fix \"**/*.ts\"",
         "format": "prettier --write \"**/*.{ts,html,css,scss,md,js,json}\"",
         "coverage": "jest --config ./jest.app.config.js --coverage && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage"
     },
@@ -38,6 +38,7 @@
         "zone.js": "~0.10.3"
     },
     "devDependencies": {
+        "@angular-ru/eslint-config": "12.20.1",
         "@angular-ru/prettier-config": "^12.19.1",
         "@angular-ru/tsconfig": "^12.19.1",
         "@angular-devkit/build-angular": "^0.803.27",

projects/logger/src/lib/decorators/autobind.decorator.ts

@@ -29,11 +29,14 @@ function autoBindClass(target: ObjectKeyMap): Any {
 function getOwnPropertyDescriptors(target: ObjectKeyMap): ObjectKeyMap {
     const descriptors: ObjectKeyMap = {};
 
-    getOwnKeys(target).forEach((key: string) => (descriptors[key] = getOwnPropertyDescriptor(target, key)));
+    getOwnKeys(target).forEach((key: string): void => {
+        descriptors[key] = getOwnPropertyDescriptor(target, key);
+    });
 
     return descriptors;
 }
 
+// eslint-disable-next-line max-lines-per-function
 function autoBindMethod(
     target: ObjectKeyMap,
     key: string,
@@ -68,6 +71,7 @@ function handle(args: Any[]): Any {
     if (args.length === 1) {
         return autoBindClass(args[0]);
     } else {
+        // eslint-disable-next-line @typescript-eslint/ban-ts-ignore
         // @ts-ignore
         return autoBindMethod(...args);
     }
@@ -84,7 +88,7 @@ export function autoBind(...args: Any[]): Any {
 }
 
 function createDefaultSetter(key: Any): Fn {
-    return function set(this: any, newValue: unknown): unknown {
+    return function set(this: Any, newValue: unknown): unknown {
         Object.defineProperty(this, key, {
             configurable: true,
             writable: true,

projects/logger/src/lib/decorators/debug.decorator.ts

@@ -1,6 +1,6 @@
+import { LogFn } from '../interfaces/logger.external';
 import { LoggerInjector } from '../logger.injector';
 import { LoggerService } from '../logger.service';
-import { LogFn } from '../interfaces/logger.external';
 
 export function DebugLog(): PropertyDecorator {
     return (target: unknown, propertyName: string | symbol): void => {

projects/logger/src/lib/decorators/error.decorator.ts

@@ -1,6 +1,6 @@
+import { LogFn } from '../interfaces/logger.external';
 import { LoggerInjector } from '../logger.injector';
 import { LoggerService } from '../logger.service';
-import { LogFn } from '../interfaces/logger.external';
 
 export function ErrorLog(): PropertyDecorator {
     return (target: unknown, propertyName: string | symbol): void => {