Merge pull request #117 from masterchief01/linter-formatter

rajitha1998 · web-flow · commit c85971922595 · 2022-06-19T21:35:40.000+05:30
Linter formatter
diff --git a/.eslintignore b/.eslintignore
@@ -0,0 +1,2 @@
+node_modules
+dist
diff --git a/.eslintrc b/.eslintrc
@@ -0,0 +1,18 @@
+{
+	"root": true,
+	"parser": "@typescript-eslint/parser",
+	"plugins": ["@typescript-eslint", "simple-import-sort"],
+	"extends": [
+		"eslint:recommended",
+		"plugin:@typescript-eslint/eslint-recommended",
+		"plugin:@typescript-eslint/recommended"
+	],
+	"rules": {
+		"prefer-const": "warn",
+		"no-unused-vars": "warn",
+		"no-async-promise-executor": "off",
+		"@typescript-eslint/no-inferrable-types": "off",
+		"simple-import-sort/imports": "error",
+		"simple-import-sort/exports": "error"
+	}
+}
diff --git a/.prettierignore b/.prettierignore
@@ -0,0 +1,3 @@
+node_modules
+dist
+coverage
diff --git a/.prettierrc b/.prettierrc
@@ -0,0 +1,7 @@
+{ 
+    "tabWidth": 4, 
+    "singleQuote": true,  
+    "printWidth": 80, 
+    "useTabs": true, 
+    "trailingComma": "es5"
+}
diff --git a/generator/.eslintrc b/generator/.eslintrc
@@ -0,0 +1,12 @@
+{
+	"extends": "../.eslintrc",
+	"ignorePatterns": [
+		"*.js",
+		"node_modules/",
+		"dummyClasses/",
+		"generatedClasses/"
+	],
+	"rules": {
+		"@typescript-eslint/no-empty-function": "warn"
+	}
+}
diff --git a/generator/generators/aws/generator.ts b/generator/generators/aws/generator.ts
@@ -1,8 +1,9 @@
 import * as fs from "fs";
 import { createSourceFile, ScriptTarget, SyntaxKind } from "typescript";
+
 import { getAST } from "../../parsers/aws/parser";
 import { transform } from "../../transformers/aws/transformer";
-import { filters, groupers, printFile, getDir } from "../lib/helper";
+import { filters, getDir,groupers, printFile } from "../lib/helper";
 
 interface FunctionData {
   functionName: string;
diff --git a/generator/generators/azure/generator.ts b/generator/generators/azure/generator.ts
@@ -1,8 +1,9 @@
 import * as fs from "fs";
-import { SyntaxKind, createSourceFile, ScriptTarget } from "typescript";
+import { createSourceFile, ScriptTarget,SyntaxKind } from "typescript";
+
 import { getAST } from "../../parsers/azure/parser";
-import { groupers, filters, printFile, getDir } from "../lib/helper";
 import { transform } from "../../transformers/azure/transformer";
+import { filters, getDir,groupers, printFile } from "../lib/helper";
 
 interface FunctionData {
   pkgName: string;
diff --git a/generator/generators/do/generator.ts b/generator/generators/do/generator.ts
@@ -1,8 +1,9 @@
 import * as fs from "fs";
 import { createSourceFile, ScriptTarget, SyntaxKind } from "typescript";
+
 import { getAST } from "../../parsers/do/parser";
 import { transform } from "../../transformers/do/transformer";
-import { printFile, getDir } from "../lib/helper";
+import { getDir,printFile } from "../lib/helper";
 
 interface FunctionData {
   functionName: string;
diff --git a/generator/generators/googleCloud/generator.ts b/generator/generators/googleCloud/generator.ts
@@ -1,10 +1,11 @@
 import * as fs from "fs";
 import * as path from "path";
-import { SyntaxKind, createSourceFile, ScriptTarget } from "typescript";
+import { createSourceFile, ScriptTarget,SyntaxKind } from "typescript";
+
 import { getAST } from "../../parsers/googleCloud/parser";
-import { filters, groupers, printFile, getDir } from "../lib/helper";
-import { clientBasedTransform } from "../../transformers/googleCloud/clientBasedTransformer";
 import { classBasedTransform } from "../../transformers/googleCloud/classBasedTransformer";
+import { clientBasedTransform } from "../../transformers/googleCloud/clientBasedTransformer";
+import { filters, getDir,groupers, printFile } from "../lib/helper";
 
 interface ClassData {
   name: string;
diff --git a/generator/generators/lib/helper.ts b/generator/generators/lib/helper.ts
@@ -1,12 +1,11 @@
 import * as fs from "fs";
 
 import { groupAWSMethods } from "../lib/aws/awsHelper";
-import { groupGCPMethods } from "../lib/googleCloud/gcpHelper";
-import { groupAzureMethods } from "../lib/azure/azureHelper";
-
 import { filterAWSMethods } from "../lib/aws/awsHelper";
-import { filterGCPMethods } from "../lib/googleCloud/gcpHelper";
+import { groupAzureMethods } from "../lib/azure/azureHelper";
 import { filterAzureMethods } from "../lib/azure/azureHelper";
+import { groupGCPMethods } from "../lib/googleCloud/gcpHelper";
+import { filterGCPMethods } from "../lib/googleCloud/gcpHelper";
 
 const dirMap = {
   appServices: ["PaaS"],
@@ -38,12 +37,12 @@ const filters = {
 };
 
 const getDir = (service: string): string => {
-  for (var dir in dirMap) {
+  for (let dir in dirMap) {
     if (dirMap[dir].includes(service)) {
       return dir;
     }
   }
   throw new Error("Not a valid service: " + service);
 };
 
-export { groupers, filters, getDir };
+export { filters, getDir,groupers };
diff --git a/generator/main.ts b/generator/main.ts
@@ -1,9 +1,10 @@
-import * as yaml from "js-yaml";
 import * as fs from "fs";
+import * as yaml from "js-yaml";
+
 import { generateAWSClass } from "./generators/aws/generator";
 import { generateAzureClass } from "./generators/azure/generator";
-import { generateGCPClass } from "./generators/googleCloud/generator";
 import { generateDOClass } from "./generators/do/generator";
+import { generateGCPClass } from "./generators/googleCloud/generator";
 
 try {
   const services = yaml.safeLoad(fs.readFileSync("node-cloud.yml", "utf8"));
diff --git a/generator/package.json b/generator/package.json
@@ -11,7 +11,9 @@
   ],
   "scripts": {
     "test": "cross-env TS_NODE_FILES=true mocha --exit --require ts-node/register --colors test/**/*.ts",
-    "tool": "tsc main && node main"
+    "tool": "tsc main && node main",
+    "lint": "eslint .",
+		"lint-fix": "eslint --fix ."
   },
   "dependencies": {
     "@azure/arm-appservice": "^6.0.0",
diff --git a/generator/test/generators/aws/generator.test.ts b/generator/test/generators/aws/generator.test.ts
@@ -1,8 +1,9 @@
 import { expect } from "chai";
-import { extractSDKData } from "../../../generators/aws/generator";
-import { readSourceFile, readJsonData } from "../lib/helper";
 import { SyntaxKind } from "typescript";
 
+import { extractSDKData } from "../../../generators/aws/generator";
+import { readJsonData,readSourceFile } from "../lib/helper";
+
 describe("AWS generator extractSDKData", () => {
   context("with valid methods and valid AST", () => {
     it("should return extracted class data", async () => {
diff --git a/generator/test/generators/azure/generator.test.ts b/generator/test/generators/azure/generator.test.ts
@@ -1,7 +1,8 @@
 import { expect } from "chai";
-import { extractSDKData } from "../../../generators/azure/generator";
 import { SyntaxKind } from "typescript";
-import { readSourceFile, readJsonData } from "../lib/helper";
+
+import { extractSDKData } from "../../../generators/azure/generator";
+import { readJsonData,readSourceFile } from "../lib/helper";
 
 describe("Azure generator extractSDKData", () => {
   context("with valid methods and valid AST", () => {
diff --git a/generator/test/generators/do/generator.test.ts b/generator/test/generators/do/generator.test.ts
@@ -1,8 +1,9 @@
 import { expect } from "chai";
-import { extractSDKData } from "../../../generators/do/generator";
-import { readSourceFile, readJsonData } from "../lib/helper";
 import { SyntaxKind } from "typescript";
 
+import { extractSDKData } from "../../../generators/do/generator";
+import { readJsonData,readSourceFile } from "../lib/helper";
+
 describe("Digital Ocean generator extractSDKData", () => {
   context("with valid methods and valid AST", () => {
     it("should return extracted class data", async () => {
diff --git a/generator/test/generators/googleCloud/generator.test.ts b/generator/test/generators/googleCloud/generator.test.ts
@@ -1,10 +1,11 @@
 import { expect } from "chai";
+import { SyntaxKind } from "typescript";
+
 import {
   extractClassBasedSDKData,
   extractClientBasedSDKdata
 } from "../../../generators/googleCloud/generator";
 import { readJsonData, readSourceFile } from "../lib/helper";
-import { SyntaxKind } from "typescript";
 
 describe("GCP generator extractClassBasedSDKData", () => {
   context("with valid methods and valid AST", () => {
diff --git a/generator/test/generators/lib/helper.ts b/generator/test/generators/lib/helper.ts
@@ -1,5 +1,5 @@
 import * as fs from "fs";
-import { ScriptTarget, createSourceFile } from "typescript";
+import { createSourceFile,ScriptTarget } from "typescript";
 
 export function readSourceFile(datasetName, provider) {
   return new Promise((resolve, reject) => {
diff --git a/generator/test/parsers/aws/parser.test.ts b/generator/test/parsers/aws/parser.test.ts
@@ -1,7 +1,8 @@
 import { expect } from "chai";
-import { getAST } from "../../../parsers/aws/parser";
 import { SyntaxKind } from "typescript";
 
+import { getAST } from "../../../parsers/aws/parser";
+
 describe("AWS parser getAST", () => {
   context("with existing file", () => {
     it("should return Abstract syntax tree of the class", async () => {
diff --git a/generator/test/parsers/azure/parser.test.ts b/generator/test/parsers/azure/parser.test.ts
@@ -1,7 +1,8 @@
 import { expect } from "chai";
-import { getAST } from "../../../parsers/azure/parser";
 import { SyntaxKind } from "typescript";
 
+import { getAST } from "../../../parsers/azure/parser";
+
 describe("Azure parser getAST", () => {
   context("with existing file", () => {
     it("should return Abstract syntax tree of the class", async () => {
diff --git a/generator/test/parsers/do/parser.test.ts b/generator/test/parsers/do/parser.test.ts
@@ -1,7 +1,8 @@
 import { expect } from "chai";
-import { getAST } from "../../../parsers/do/parser";
 import { SyntaxKind } from "typescript";
 
+import { getAST } from "../../../parsers/do/parser";
+
 describe("Digital Ocean parser getAST", () => {
   context("With existing file", () => {
     it("Should return Abstract syntax tree of the class", async () => {
diff --git a/generator/test/parsers/googleCloud/parser.test.ts b/generator/test/parsers/googleCloud/parser.test.ts
@@ -1,7 +1,8 @@
 import { expect } from "chai";
-import { getAST } from "../../../parsers/googleCloud/parser";
 import { SyntaxKind } from "typescript";
 
+import { getAST } from "../../../parsers/googleCloud/parser";
+
 describe("Google cloud parser getAST", () => {
   context("with existing file, multi:false", () => {
     it("should return Abstract syntax tree of the class", async () => {
diff --git a/generator/test/transformers/aws/transformer.test.ts b/generator/test/transformers/aws/transformer.test.ts
@@ -1,6 +1,7 @@
 import { expect } from "chai";
+import { createSourceFile, isSourceFile,ScriptTarget } from "typescript";
+
 import { transform } from "../../../transformers/aws/transformer";
-import { createSourceFile, ScriptTarget, isSourceFile } from "typescript";
 import { readJsonData, readSourceFile } from "../lib/helper";
 
 interface TestData {
diff --git a/generator/test/transformers/azure/transformer.test.ts b/generator/test/transformers/azure/transformer.test.ts
@@ -1,6 +1,7 @@
 import { expect } from "chai";
+import { createSourceFile, isSourceFile,ScriptTarget } from "typescript";
+
 import { transform } from "../../../transformers/azure/transformer";
-import { createSourceFile, ScriptTarget, isSourceFile } from "typescript";
 import { readJsonData, readSourceFile } from "../lib/helper";
 
 interface TestData {
diff --git a/generator/test/transformers/do/transformer.test.ts b/generator/test/transformers/do/transformer.test.ts
@@ -1,6 +1,7 @@
 import { expect } from "chai";
+import { createSourceFile, isSourceFile,ScriptTarget } from "typescript";
+
 import { transform } from "../../../transformers/do/transformer";
-import { createSourceFile, ScriptTarget, isSourceFile } from "typescript";
 import { readJsonData, readSourceFile } from "../lib/helper";
 
 interface TestData {
diff --git a/generator/test/transformers/googleCloud/classBasedTransformer.test.ts b/generator/test/transformers/googleCloud/classBasedTransformer.test.ts
@@ -1,6 +1,7 @@
 import { expect } from "chai";
+import { createSourceFile, isSourceFile,ScriptTarget } from "typescript";
+
 import { classBasedTransform } from "../../../transformers/googleCloud/classBasedTransformer";
-import { createSourceFile, ScriptTarget, isSourceFile } from "typescript";
 import { readJsonData, readSourceFile } from "../lib/helper";
 
 interface TestData {
diff --git a/generator/test/transformers/googleCloud/clientBasedTransformer.test.ts b/generator/test/transformers/googleCloud/clientBasedTransformer.test.ts
@@ -1,6 +1,7 @@
 import { expect } from "chai";
+import { createSourceFile, isSourceFile,ScriptTarget } from "typescript";
+
 import { clientBasedTransform } from "../../../transformers/googleCloud/clientBasedTransformer";
-import { createSourceFile, ScriptTarget, isSourceFile } from "typescript";
 import { readJsonData, readSourceFile } from "../lib/helper";
 
 interface TestData {
diff --git a/generator/test/transformers/lib/helper.ts b/generator/test/transformers/lib/helper.ts
@@ -1,5 +1,5 @@
 import * as fs from "fs";
-import { ScriptTarget, createSourceFile } from "typescript";
+import { createSourceFile,ScriptTarget } from "typescript";
 
 export function readSourceFile(datasetName, provider) {
   return new Promise((resolve, reject) => {
diff --git a/generator/transformers/aws/transformer.ts b/generator/transformers/aws/transformer.ts
@@ -1,5 +1,5 @@
-import * as ts from "typescript";
 import { cloneDeep } from "lodash";
+import * as ts from "typescript";
 
 const dummyIdentifiers = [
   "ClassName",
diff --git a/generator/transformers/azure/transformer.ts b/generator/transformers/azure/transformer.ts
@@ -1,5 +1,5 @@
-import * as ts from "typescript";
 import { cloneDeep } from "lodash";
+import * as ts from "typescript";
 
 const dummyIdentifiers = [
   "ClassName",
diff --git a/generator/transformers/do/transformer.ts b/generator/transformers/do/transformer.ts
@@ -1,5 +1,5 @@
-import * as ts from "typescript";
 import { cloneDeep } from "lodash";
+import * as ts from "typescript";
 
 const dummyIdentifiers = [
   "ClassName",
diff --git a/generator/transformers/googleCloud/classBasedTransformer.ts b/generator/transformers/googleCloud/classBasedTransformer.ts
@@ -1,5 +1,5 @@
-import * as ts from "typescript";
 import { cloneDeep } from "lodash";
+import * as ts from "typescript";
 
 const dummyIdentifiers = [
   "ClassName",
diff --git a/generator/transformers/googleCloud/clientBasedTransformer.ts b/generator/transformers/googleCloud/clientBasedTransformer.ts
@@ -1,5 +1,5 @@
-import * as ts from "typescript";
 import { cloneDeep } from "lodash";
+import * as ts from "typescript";
 
 const dummyIdentifiers = [
   "ClassName",
diff --git a/package.json b/package.json
@@ -37,9 +37,13 @@
     "@types/lodash": "^4.14.158",
     "@types/mocha": "^8.0.0",
     "@types/node": "^14.0.6",
+    "@typescript-eslint/eslint-plugin": "^5.19.0",
+    "@typescript-eslint/parser": "^5.19.0",
     "better-docs": "^2.3.0",
     "chai": "^4.2.0",
     "cross-env": "^7.0.2",
+    "eslint": "^8.13.0",
+    "eslint-plugin-simple-import-sort": "^7.0.0",
     "husky": "^4.3.0",
     "jsdoc": "^3.6.5",
     "lerna": "^3.22.1",
@@ -51,7 +55,7 @@
   },
   "husky": {
     "hooks": {
-      "pre-commit": "lerna run test && lerna run prettier",
+      "pre-commit": "lerna run test && lerna run prettier && lerna run lint",
       "pre-push": "lerna run test"
     }
   }
diff --git a/yarn.lock b/yarn.lock
