Do some linting

PythonCoderAS · PythonCoderAS · commit a41d9f037238 · 2022-06-28T18:45:27.000-04:00
diff --git a/.eslintrc b/.eslintrc
@@ -1,3 +1,7 @@
 {
-  "extends": ["pythoncoderas-combo"]
+  "extends": ["pythoncoderas-combo"],
+  "rules": {
+    "camelcase": "off",
+    "no-param-reassign": "off"
+  }
 }
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -38,7 +38,6 @@
     "chai": "^4.3.6",
     "chai-as-promised": "^7.1.1",
     "eslint": "^8.17.0",
-    "eslint-config-prettier": "^8.5.0",
     "eslint-config-pythoncoderas-combo": "^1.0.1",
     "mocha": "^10.0.0",
     "prettier": "^2.6.2",
diff --git a/src/handler.ts b/src/handler.ts
@@ -17,7 +17,7 @@ async function spawn(command: string, args?: string[], options?: object) {
   return childProcess;
 }
 
-// eslint-disable-next-line no-undef -- NodeJS is a fake namespace by TypeScript.
+
 function terminateSpawnCache(signal: NodeJS.Signals = "SIGTERM") {
   for (const childProcess of spawnedProcesses.keys()) {
     childProcess.kill(signal);
@@ -118,7 +118,7 @@ export default async function handler(script: string, flags: CliFlags) {
     );
   } finally {
     if (flags.terminate ?? true) {
-      // eslint-disable-next-line no-undef -- NodeJS is a fake namespace by TypeScript.
+
       terminateSpawnCache(flags.signal as NodeJS.Signals);
     }
 
diff --git a/src/template.ts b/src/template.ts
@@ -1,14 +1,14 @@
-import { Template } from "./types";
 import { Sade } from "sade";
+import { Template } from "./types";
 import templates from "./templates";
 
 function generateCLIFromTemplate(sade: Sade, template: Template): Sade {
   const argString: string = template.arguments
-    ? " " +
+    ? ` ${
       template.arguments
         .map((arg) => (typeof arg === "string" ? { name: arg } : arg))
         .map((arg) => `<${arg.name}>`)
-        .join(" ")
+        .join(" ")}`
     : "";
   sade = sade.command(
     `template ${template.name}${argString}`,
@@ -37,5 +37,6 @@ export default function main(sade: Sade): Sade {
   for (const template of templates) {
     sade = generateCLIFromTemplate(sade, template);
   }
+
   return sade;
 }
diff --git a/src/templates/cp.ts b/src/templates/cp.ts
@@ -1,11 +1,13 @@
+import { writeFile } from "fs/promises";
 import { GenerateTemplateCliFlags, Template } from "../types";
 import { getOutputPath } from "../utils";
-import { writeFile } from "fs/promises";
 
 export default class CpTemplate implements Template {
   readonly name = "copy";
+
   readonly description =
     "A template to copy a signle source file to multiple repositories and commit the changes.";
+
   readonly aliases = ["cp"];
 
   readonly defaultFileName = "cp.sh";
diff --git a/src/types.ts b/src/types.ts
@@ -27,7 +27,9 @@ export interface Template {
   defaultFileName: string;
 
   arguments?: (string | { name: string; description?: string })[];
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   flags?: (string | { name: string; description?: string; value?: any })[];
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   handler: (...args: any[]) => any;
 }
 
diff --git a/src/utils.ts b/src/utils.ts
@@ -1,7 +1,7 @@
 import { ChildProcess } from "child_process";
-import { GenerateTemplateCliFlags, RepoOwner, Template } from "./types";
 import { mkdtemp } from "fs/promises";
 import { tmpdir } from "os";
+import { GenerateTemplateCliFlags, RepoOwner, Template } from "./types";
 
 export function getRepoAndOwner(
   input: string,

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,7 @@`
`1`	`1`	`{`
`2`		`- "extends": ["pythoncoderas-combo"]`
	`2`	`+ "extends": ["pythoncoderas-combo"],`
	`3`	`+ "rules": {`
	`4`	`+ "camelcase": "off",`
	`5`	`+ "no-param-reassign": "off"`
	`6`	`+ }`
`3`	`7`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,9 @@ export interface Template {`
`27`	`27`	`defaultFileName: string;`
`28`	`28`
`29`	`29`	`arguments?: (string \| { name: string; description?: string })[];`
	`30`	`+ // eslint-disable-next-line @typescript-eslint/no-explicit-any`
`30`	`31`	`flags?: (string \| { name: string; description?: string; value?: any })[];`
	`32`	`+ // eslint-disable-next-line @typescript-eslint/no-explicit-any`
`31`	`33`	`handler: (...args: any[]) => any;`
`32`	`34`	`}`
`33`	`35`