Merge pull request #410 from technote-space/release/next-v5.3.2

release: v5.3.3

Author: technote-space

.eslintrc

@@ -109,7 +109,8 @@
     "sort-imports": 0,
     "import/order": [2, {
       "groups": ["type", "builtin", "external", "internal", "parent", "sibling", "index", "object"],
-      "alphabetize": { "order": "asc", "caseInsensitive": true },
-    }]
+      "alphabetize": { "order": "asc", "caseInsensitive": true }
+    }],
+    "@typescript-eslint/no-non-null-assertion": "off"
   }
 }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@technote-space/github-action-helper",
-  "version": "5.3.2",
+  "version": "5.3.3",
   "description": "Helper for GitHub Action.",
   "keywords": [
     "github",
@@ -44,7 +44,7 @@
     "@actions/github": "^5.0.1",
     "@octokit/openapi-types": "^11.2.0",
     "@octokit/plugin-rest-endpoint-methods": "^5.13.0",
-    "@technote-space/github-action-log-helper": "^0.2.1",
+    "@technote-space/github-action-log-helper": "^0.2.2",
     "shell-escape": "^0.2.0",
     "sprintf-js": "^1.1.2"
   },
@@ -53,7 +53,7 @@
     "@commitlint/config-conventional": "^16.2.1",
     "@rollup/plugin-typescript": "^8.3.2",
     "@sindresorhus/tsconfig": "^2.0.0",
-    "@technote-space/github-action-test-helper": "^0.9.2",
+    "@technote-space/github-action-test-helper": "^0.9.3",
     "@types/node": "^17.0.25",
     "@types/shell-escape": "^0.2.0",
     "@typescript-eslint/eslint-plugin": "^5.20.0",

src/api-helper.ts

@@ -122,7 +122,7 @@ export default class ApiHelper {
       }));
     }
 
-    return this.prCache[key];
+    return this.prCache[key]!;
   };
 
   public filesToBlobs = async(rootDir: string, files: Array<string>): Promise<Array<{ path: string; sha: string }>> => await Promise.all(files.map(file => this.createBlob(rootDir, file)));
@@ -198,7 +198,7 @@ export default class ApiHelper {
       head: `${this.context.repo.owner}:${getBranch(branchName, false)}`,
     });
     if (response.data.length) {
-      return response.data[0];
+      return response.data[0]!;
     }
 
     return null;

src/git-helper.test.ts

@@ -1,6 +1,6 @@
 /* eslint-disable no-magic-numbers */
-import { ExecException } from 'child_process';
-import { Context } from '@actions/github/lib/context';
+import type { Context } from '@actions/github/lib/context';
+import type { ExecException } from 'child_process';
 import { Logger } from '@technote-space/github-action-log-helper';
 import {
   testEnv,

src/git-helper.ts

@@ -130,7 +130,7 @@ export default class GitHelper {
       args: ['--abbrev-ref', 'HEAD'],
       suppressError: this.shouldSuppressError(),
       stderrToStdout: true,
-    }))[0].stdout[0]?.trim() ?? '';
+    }))[0]!.stdout[0]?.trim() ?? '';
   };
 
   public cloneBranch = async(workDir: string, branch: string, context: Context): Promise<void> => {
@@ -271,7 +271,7 @@ export default class GitHelper {
     command: 'git status',
     args: ['--short', '-uno'],
     suppressOutput: true,
-  }))[0].stdout.filter(line => line.match(/^[MDA]\s+/)).filter(this.filter).map(line => line.replace(/^[MDA]\s+/, ''));
+  }))[0]!.stdout.filter(line => line.match(/^[MDA]\s+/)).filter(this.filter).map(line => line.replace(/^[MDA]\s+/, ''));
 
   public getRefDiff = async(workDir: string, baseRef: string, compareRef: string, diffFilter?: string, dot?: '..' | '...'): Promise<string[]> => {
     const toDiffRef = (ref: string): string =>
@@ -282,7 +282,7 @@ export default class GitHelper {
       command: 'git diff',
       args: [`${toDiffRef(baseRef)}${dot ?? '...'}${toDiffRef(compareRef)}`, '--name-only', diffFilter ? `--diff-filter=${diffFilter}` : ''],
       suppressOutput: true,
-    }))[0].stdout.filter(item => !!item.trim());
+    }))[0]!.stdout.filter(item => !!item.trim());
   };
 
   public checkDiff = async(workDir: string): Promise<boolean> => !!(await this.getDiff(workDir)).length;
@@ -320,7 +320,7 @@ export default class GitHelper {
     command: 'git tag',
     suppressOutput: options?.suppressOutput || options?.quiet,
     altCommand: options?.quiet ? '' : undefined,
-  }))[0].stdout;
+  }))[0]!.stdout;
 
   public fetchTags = async(workDir: string, context: Context, splitSize = 20): Promise<void> => { // eslint-disable-line no-magic-numbers
     await this.runCommand(workDir, [

src/utils.ts

@@ -46,7 +46,7 @@ export const parseVersion = (version: string, options?: { fill?: boolean; cut?:
     return undefined;
   }
 
-  const fragments = split(matches[1], '.');
+  const fragments = split(matches[1]!, '.');
   // eslint-disable-next-line no-magic-numbers
   const length    = options?.slice && options.slice < 0 ? (options.length ?? 3) : (options?.slice ?? options?.length ?? 3);
   // eslint-disable-next-line no-magic-numbers
@@ -220,7 +220,7 @@ export const versionCompare = (version1: string, version2: string, checkDifferen
     const val1 = version1[num] ?? (checkDifferentLevel ? 0 : version2[num]);
     // eslint-disable-next-line no-magic-numbers
     const val2 = version2[num] ?? (checkDifferentLevel ? 0 : version1[num]);
-    return val1 === val2 ? compare(version1, version2, ++num) : Math.sign(val1 - val2);
+    return val1 === val2 ? compare(version1, version2, ++num) : Math.sign(val1! - val2!);
   };
   return compare(splitVersion(version1.replace(/^v/, '')), splitVersion(version2.replace(/^v/, '')));
 };
@@ -261,17 +261,17 @@ export const isOutputDebug = (): boolean => getInput('UTILS_OUTPUT_DEBUG') === '
 export const objectGet = <T>(value: Record<string, any> | undefined | null, key: string, defaultValue?: T): T | undefined => {
   const keys = key.split('.');
 
-  if (!keys.length || !value || !(keys[0] in value)) {
+  if (!keys.length || !value || !(keys[0]! in value)) {
     return defaultValue;
   }
 
   // eslint-disable-next-line no-magic-numbers
   if (keys.length > 1) {
     // eslint-disable-next-line no-magic-numbers
-    return objectGet(value[keys[0]], keys.slice(1).join('.'), defaultValue);
+    return objectGet(value[keys[0]!], keys.slice(1).join('.'), defaultValue);
   }
 
-  return value[keys[0]];
+  return value[keys[0]!];
 };
 
 export const ensureNotNullValue = <T>(value: T | null | undefined, defaultValue: T): T => value ?? defaultValue;

tsconfig.json

@@ -7,8 +7,7 @@
       "es2020"
     ],
     "noPropertyAccessFromIndexSignature": false,
-    "noImplicitAny": false,
-    "strictNullChecks": false
+    "noImplicitAny": false
   },
   "include": [
     "src"

yarn.lock

@@ -379,18 +379,18 @@
   resolved "https://registry.yarnpkg.com/@sindresorhus/tsconfig/-/tsconfig-2.0.0.tgz#610e1cb8ed4e1cb7ff0cb4cb8712cd607315c152"
   integrity sha512-1nsyWtFtPpVjEnNzp2rd03BVz84BYHeW7LNYvmcj3q4jAgJL2hl0XBp3WmDFFPjFLF3w+L0XEYqhjWUTU+TCzw==
 
-"@technote-space/github-action-log-helper@^0.2.1":
-  version "0.2.1"
-  resolved "https://registry.yarnpkg.com/@technote-space/github-action-log-helper/-/github-action-log-helper-0.2.1.tgz#5319d9f13add3bbaa75be9c1422cf9c986655b9f"
-  integrity sha512-IakrjNmMiaYKOcrk/id+oqvGzGXofcIYlpCUngyzOyVrsXCd2o7b/8eTSHAknGg8o82lN6KLW99Su9iIJye3hg==
+"@technote-space/github-action-log-helper@^0.2.2":
+  version "0.2.2"
+  resolved "https://registry.yarnpkg.com/@technote-space/github-action-log-helper/-/github-action-log-helper-0.2.2.tgz#90bf784372a4af0fb720778b81c3b75de9b5cafa"
+  integrity sha512-5jHDxczTLgC4tCJ8Xc7+g1l3N1rEjfogYJiYMUA16a0okD27x9Ymq6pnlDsV6zfsq6bTlTBEHFWLFJgjVKkZpg==
   dependencies:
     "@actions/core" "^1.6.0"
     sprintf-js "^1.1.2"
 
-"@technote-space/github-action-test-helper@^0.9.2":
-  version "0.9.2"
-  resolved "https://registry.yarnpkg.com/@technote-space/github-action-test-helper/-/github-action-test-helper-0.9.2.tgz#8721f18658136f3c8ce27819940293d0847daec5"
-  integrity sha512-2kuw9cvI+sn768OAAtkIVZoPOqPZUptvsMlEURkG6mFWCtifzY0Qj52rzX9vuJWLEuwRu6JSldBiJnjepXmATQ==
+"@technote-space/github-action-test-helper@^0.9.3":
+  version "0.9.3"
+  resolved "https://registry.yarnpkg.com/@technote-space/github-action-test-helper/-/github-action-test-helper-0.9.3.tgz#b7e4e3bce6ebd0543000740ac297947e44de8b68"
+  integrity sha512-8h97sKU9rNteyErnzfTimnUifVE6SFa3HebzPWWeEIjt7KPPFCXrSaAIx1o+4Es1In46oOLZQRJOK1fwSKEaYg==
   dependencies:
     "@actions/core" "^1.6.0"
     "@actions/github" "^5.0.1"