fix(examples): fix scope error in cli.ts

Found by @HaveAGitGat: #136 (comment) Enable eslint no-shadow rule.
buzz · Jul 31, 2023 · cd044ff · cd044ff
1 parent 48cdcc3
commit cd044ff
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -30,6 +30,8 @@
     // File extensions
     "import/extensions": ["error", "never"],
 
+    "no-shadow": "warn",
+
     "prettier/prettier": "error"
   },
   "overrides": [

diff --git a/src/cli.ts b/src/cli.ts
@@ -33,7 +33,7 @@ const analyze = async ({ coverData, file, format, full }: ReturnType<typeof pars
   try {
     fileHandle = await fsPromises.open(file, 'r')
     fileSize = (await fileHandle.stat()).size
-    const mediainfo = await MediaInfoFactory({ format, coverData, full })
+    mediainfo = await MediaInfoFactory({ format, coverData, full })
     if (mediainfo === undefined) {
       throw new Error('Failed to initialize MediaInfo')
     }
@@ -68,9 +68,9 @@ function parseArgs() {
     .positional('file', { describe: 'File to analyze', type: 'string' })
     .help('h')
     .alias('h', 'help')
-    .fail(function (msg: string, err: Error, yargs) {
+    .fail((msg: string, err: Error, argv) => {
       if (msg) {
-        console.error(yargs.help())
+        console.error(argv.help())
         console.error(msg)
       }
       if (err) {