Update dependency prettier to v3 (postcss#454)

renovate[bot] · RyanZim · web-flow · commit d8b2e2af3716 · 2023-07-15T20:53:50.000-04:00
* Update dependency prettier to v3

* Run prettier

---------

Co-authored-by: renovate[bot] &lt;29139614+renovate[bot]@users.noreply.github.com&gt;
Co-authored-by: Ryan Zimmerman &lt;opensrc@ryanzim.com&gt;
diff --git a/index.js b/index.js
@@ -50,7 +50,7 @@ async function buildCliConfig() {
               if (e.name && e.name !== 'Error') prefix += `: ${e.name}`
               return error(`Plugin Error${prefix}: ${msg}'`)
             }
-          })
+          }),
         )
       : [],
   }
@@ -88,13 +88,13 @@ buildCliConfig()
     if (input && input.length) {
       return globby(
         input.map((i) => slash(String(i))),
-        { dot: argv.includeDotfiles }
+        { dot: argv.includeDotfiles },
       )
     }
 
     if (argv.replace || argv.dir) {
       error(
-        'Input Error: Cannot use --dir or --replace when reading from stdin'
+        'Input Error: Cannot use --dir or --replace when reading from stdin',
       )
     }
 
@@ -111,7 +111,7 @@ buildCliConfig()
 
     if (i.length > 1 && !argv.dir && !argv.replace) {
       error(
-        'Input Error: Must use --dir or --replace with multiple input files'
+        'Input Error: Must use --dir or --replace with multiple input files',
       )
     }
 
@@ -146,7 +146,7 @@ buildCliConfig()
           .concat(getAncestorDirs(file).flatMap(depGraph.dependantsOf))
 
         recompile = recompile.concat(
-          dependants.filter((file) => input.includes(file))
+          dependants.filter((file) => input.includes(file)),
         )
 
         if (!recompile.length) recompile = input
@@ -171,7 +171,7 @@ function rc(ctx, path) {
     .then((rc) => {
       if (rc.options.from || rc.options.to) {
         error(
-          'Config Error: Can not set from or to options in config file, use CLI arguments instead'
+          'Config Error: Can not set from or to options in config file, use CLI arguments instead',
         )
       }
       configFile = rc.file
@@ -195,7 +195,7 @@ function files(files) {
       }
 
       return read(file).then((content) => css(content, file))
-    })
+    }),
   )
 }
 
@@ -246,7 +246,7 @@ function css(css, file) {
 
       if (!options.to && config.options.map && !config.options.map.inline) {
         error(
-          'Output Error: Cannot output external sourcemaps when writing to STDOUT'
+          'Output Error: Cannot output external sourcemaps when writing to STDOUT',
         )
       }
 
@@ -268,8 +268,8 @@ function css(css, file) {
             const prettyTime = prettyHrtime(process.hrtime(time))
             printVerbose(
               pc.green(
-                `Finished ${pc.bold(relativePath)} in ${pc.bold(prettyTime)}`
-              )
+                `Finished ${pc.bold(relativePath)} in ${pc.bold(prettyTime)}`,
+              ),
             )
 
             const messages = result.warnings()
@@ -303,15 +303,15 @@ function dependencies(results) {
 
     result.messages
       .filter((msg) =>
-        msg.type === 'dependency' || msg.type === 'dir-dependency' ? msg : ''
+        msg.type === 'dependency' || msg.type === 'dir-dependency' ? msg : '',
       )
       .map(depGraph.add)
       .forEach((dependency) => {
         if (dependency.type === 'dir-dependency') {
           messages.push(
             dependency.glob
               ? path.join(dependency.dir, dependency.glob)
-              : dependency.dir
+              : dependency.dir,
           )
         } else {
           messages.push(dependency.file)
diff --git a/lib/args.js b/lib/args.js
@@ -7,11 +7,11 @@ const { argv } = yargs(process.argv.slice(2))
   $0 <input.css>... [OPTIONS] --dir <output-directory> [--watch|-w]
   $0 <input-directory> [OPTIONS] --dir <output-directory> [--watch|-w]
   $0 <input-glob-pattern> [OPTIONS] --dir <output-directory> [--watch|-w]
-  $0 <input.css>... [OPTIONS] --replace`
+  $0 <input.css>... [OPTIONS] --replace`,
   )
   .group(
     ['o', 'd', 'r', 'map', 'no-map', 'watch', 'verbose', 'env'],
-    'Basic options:'
+    'Basic options:',
   )
   .option('o', {
     alias: 'output',
@@ -50,7 +50,7 @@ const { argv } = yargs(process.argv.slice(2))
   })
   .group(
     ['u', 'parser', 'stringifier', 'syntax'],
-    'Options for use without a config file:'
+    'Options for use without a config file:',
   )
   .option('u', {
     alias: 'use',
@@ -98,7 +98,7 @@ const { argv } = yargs(process.argv.slice(2))
   .example('$0 src/**/*.css --base src --dir build', 'Glob Pattern & output')
   .example(
     'cat input.css | $0 -u autoprefixer > output.css',
-    'Piping input & output'
+    'Piping input & output',
   )
   .epilog(
     `If no input files are passed, it reads from stdin. If neither -o, --dir, or --replace is passed, it writes to stdout.
@@ -107,7 +107,7 @@ If there are multiple input files, the --dir or --replace option must be passed.
 
 Input files may contain globs (e.g. src/**/*.css). If you pass an input directory, it will process all files in the directory and any subdirectories, respecting the glob pattern.
 
-For more details, please see https://github.com/postcss/postcss-cli`
+For more details, please see https://github.com/postcss/postcss-cli`,
   )
 
 if (argv.ext && argv.ext.indexOf('.') !== 0) argv.ext = `.${argv.ext}`
diff --git a/package.json b/package.json
@@ -39,7 +39,7 @@
     "eslint-config-problems": "7.0.1",
     "postcss": "^8.0.4",
     "postcss-import": "^15.0.0",
-    "prettier": "~2.8.0",
+    "prettier": "~3.0.0",
     "sugarss": "^4.0.0",
     "uuid": "^9.0.0"
   },
diff --git a/test/base.js b/test/base.js
@@ -21,11 +21,11 @@ test('--base --dir works', async (t) => {
 
   t.is(
     await read(path.join(dir, 'level-1/level-2/a.css')),
-    await read('test/fixtures/base/level-1/level-2/a.css')
+    await read('test/fixtures/base/level-1/level-2/a.css'),
   )
 
   t.is(
     await read(path.join(dir, 'level-1/b.css')),
-    await read('test/fixtures/base/level-1/b.css')
+    await read('test/fixtures/base/level-1/b.css'),
   )
 })
diff --git a/test/config.js b/test/config.js
@@ -17,14 +17,14 @@ test('supports common config', async (t) => {
 
   const { error, stderr } = await cli(
     ['a.css', '-o', 'output.css', '--no-map'],
-    dir
+    dir,
   )
 
   t.falsy(error, stderr)
 
   t.is(
     await read(path.join(dir, 'output.css')),
-    await read('test/fixtures/a.css')
+    await read('test/fixtures/a.css'),
   )
 })
 
@@ -42,14 +42,14 @@ test('supports ESM config', async (t) => {
 
   const { error, stderr } = await cli(
     ['a.css', '-o', 'output.css', '--no-map'],
-    dir
+    dir,
   )
 
   t.falsy(error, stderr)
 
   t.is(
     await read(path.join(dir, 'output.css')),
-    await read('test/fixtures/a.css')
+    await read('test/fixtures/a.css'),
   )
 })
 
@@ -60,14 +60,14 @@ test("doesn't error on empty config", async (t) => {
 
   const { error, stderr } = await cli(
     ['a.css', '-o', 'output.css', '--no-map'],
-    dir
+    dir,
   )
 
   t.falsy(error, stderr)
 
   t.is(
     await read(path.join(dir, 'output.css')),
-    await read('test/fixtures/a.css')
+    await read('test/fixtures/a.css'),
   )
 })
 
@@ -82,7 +82,7 @@ test('errors if `to` is set', async (t) => {
 
   t.regex(
     stderr,
-    /Config Error: Can not set from or to options in config file, use CLI arguments instead/
+    /Config Error: Can not set from or to options in config file, use CLI arguments instead/,
   )
 })
 
@@ -97,6 +97,6 @@ test('errors if `from` is set', async (t) => {
 
   t.regex(
     stderr,
-    /Config Error: Can not set from or to options in config file, use CLI arguments instead/
+    /Config Error: Can not set from or to options in config file, use CLI arguments instead/,
   )
 })
diff --git a/test/error.js b/test/error.js
@@ -22,7 +22,7 @@ test('--map && writing to stdout', (t) => {
     t.is(code, 1, 'expected non-zero error code')
     t.regex(
       error.toString(),
-      /Output Error: Cannot output external sourcemaps when writing to STDOUT/
+      /Output Error: Cannot output external sourcemaps when writing to STDOUT/,
     )
   })
 })
@@ -33,9 +33,9 @@ test('plugin not found', (t) => {
       t.is(code, 1, 'expected non-zero error code')
       t.regex(
         error.toString(),
-        /Plugin Error: Cannot find package 'postcss-plugin'/
+        /Plugin Error: Cannot find package 'postcss-plugin'/,
       )
-    }
+    },
   )
 })
 
@@ -58,8 +58,8 @@ test('CssSyntaxError', (t) => {
       t.is(code, 1, 'expected non-zero error code')
       t.regex(
         error.toString(),
-        /CssSyntaxError: .*a.css:1:4: Unnecessary curly bracket/
+        /CssSyntaxError: .*a.css:1:4: Unnecessary curly bracket/,
       )
-    }
+    },
   )
 })
diff --git a/test/glob.js b/test/glob.js
@@ -19,14 +19,14 @@ test('works with glob patterns', async (t) => {
 
   t.is(
     await read(path.join(output, 'a.css')),
-    await read('test/fixtures/glob/a.css')
+    await read('test/fixtures/glob/a.css'),
   )
   t.is(
     await read(path.join(output, 'b.css')),
-    await read('test/fixtures/glob/b.css')
+    await read('test/fixtures/glob/b.css'),
   )
   t.is(
     await read(path.join(output, 's.css')),
-    await read('test/fixtures/glob/s.css')
+    await read('test/fixtures/glob/s.css'),
   )
 })
diff --git a/test/helpers/cli.js b/test/helpers/cli.js
@@ -13,7 +13,7 @@ export default function (args, cwd) {
           stdout,
           stderr,
         })
-      }
+      },
     )
   })
 }
diff --git a/test/helpers/read.js b/test/helpers/read.js
@@ -2,6 +2,6 @@ import fs from 'fs-extra'
 
 export default function (path) {
   return fs.readFile(path, 'utf8').then(
-    (content) => content.replace(/\r\n/g, '\n') // normalize line endings on Windows
+    (content) => content.replace(/\r\n/g, '\n'), // normalize line endings on Windows
   )
 }
diff --git a/test/misc.js b/test/misc.js
@@ -17,6 +17,6 @@ test('--version', async (t) => {
 
   t.truthy(
     version.stdout.length > 5,
-    'expected --version to output version info'
+    'expected --version to output version info',
   )
 })
diff --git a/test/stdin.js b/test/stdin.js
@@ -21,7 +21,7 @@ test.cb('reads from stdin', (t) => {
           t.end()
         })
         .catch(t.end)
-    }
+    },
   )
 
   fs.createReadStream('test/fixtures/a.css').pipe(cp.stdin)
diff --git a/test/stdout.js b/test/stdout.js
@@ -9,7 +9,7 @@ import read from './helpers/read.js'
 test.cb('writes to stdout', (t) => {
   const cp = exec(
     `node ${path.resolve(
-      'index.js'
+      'index.js',
     )} --parser sugarss -u postcss-import --no-map`,
     (error, stdout, stderr) => {
       if (error) t.end(error, stderr)
@@ -20,7 +20,7 @@ test.cb('writes to stdout', (t) => {
           t.end()
         })
         .catch(t.end)
-    }
+    },
   )
 
   fs.createReadStream('./test/fixtures/a.sss').pipe(cp.stdin)
diff --git a/test/watch.js b/test/watch.js

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ async function buildCliConfig() {`
`50`	`50`	if (e.name && e.name !== 'Error') prefix += `: ${e.name}`
`51`	`51`	return error(`Plugin Error${prefix}: ${msg}'`)
`52`	`52`	`}`
`53`		`- })`
	`53`	`+ }),`
`54`	`54`	`)`
`55`	`55`	`: [],`
`56`	`56`	`}`
`@@ -88,13 +88,13 @@ buildCliConfig()`
`88`	`88`	`if (input && input.length) {`
`89`	`89`	`return globby(`
`90`	`90`	`input.map((i) => slash(String(i))),`
`91`		`- { dot: argv.includeDotfiles }`
	`91`	`+ { dot: argv.includeDotfiles },`
`92`	`92`	`)`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`if (argv.replace \|\| argv.dir) {`
`96`	`96`	`error(`
`97`		`- 'Input Error: Cannot use --dir or --replace when reading from stdin'`
	`97`	`+ 'Input Error: Cannot use --dir or --replace when reading from stdin',`
`98`	`98`	`)`
`99`	`99`	`}`
`100`	`100`
`@@ -111,7 +111,7 @@ buildCliConfig()`
`111`	`111`
`112`	`112`	`if (i.length > 1 && !argv.dir && !argv.replace) {`
`113`	`113`	`error(`
`114`		`- 'Input Error: Must use --dir or --replace with multiple input files'`
	`114`	`+ 'Input Error: Must use --dir or --replace with multiple input files',`
`115`	`115`	`)`
`116`	`116`	`}`
`117`	`117`
`@@ -146,7 +146,7 @@ buildCliConfig()`
`146`	`146`	`.concat(getAncestorDirs(file).flatMap(depGraph.dependantsOf))`
`147`	`147`
`148`	`148`	`recompile = recompile.concat(`
`149`		`- dependants.filter((file) => input.includes(file))`
	`149`	`+ dependants.filter((file) => input.includes(file)),`
`150`	`150`	`)`
`151`	`151`
`152`	`152`	`if (!recompile.length) recompile = input`
`@@ -171,7 +171,7 @@ function rc(ctx, path) {`
`171`	`171`	`.then((rc) => {`
`172`	`172`	`if (rc.options.from \|\| rc.options.to) {`
`173`	`173`	`error(`
`174`		`- 'Config Error: Can not set from or to options in config file, use CLI arguments instead'`
	`174`	`+ 'Config Error: Can not set from or to options in config file, use CLI arguments instead',`
`175`	`175`	`)`
`176`	`176`	`}`
`177`	`177`	`configFile = rc.file`
`@@ -195,7 +195,7 @@ function files(files) {`
`195`	`195`	`}`
`196`	`196`
`197`	`197`	`return read(file).then((content) => css(content, file))`
`198`		`- })`
	`198`	`+ }),`
`199`	`199`	`)`
`200`	`200`	`}`
`201`	`201`
`@@ -246,7 +246,7 @@ function css(css, file) {`
`246`	`246`
`247`	`247`	`if (!options.to && config.options.map && !config.options.map.inline) {`
`248`	`248`	`error(`
`249`		`- 'Output Error: Cannot output external sourcemaps when writing to STDOUT'`
	`249`	`+ 'Output Error: Cannot output external sourcemaps when writing to STDOUT',`
`250`	`250`	`)`
`251`	`251`	`}`
`252`	`252`
`@@ -268,8 +268,8 @@ function css(css, file) {`
`268`	`268`	`const prettyTime = prettyHrtime(process.hrtime(time))`
`269`	`269`	`printVerbose(`
`270`	`270`	`pc.green(`
`271`		- `Finished ${pc.bold(relativePath)} in ${pc.bold(prettyTime)}`
`272`		`- )`
	`271`	+ `Finished ${pc.bold(relativePath)} in ${pc.bold(prettyTime)}`,
	`272`	`+ ),`
`273`	`273`	`)`
`274`	`274`
`275`	`275`	`const messages = result.warnings()`
`@@ -303,15 +303,15 @@ function dependencies(results) {`
`303`	`303`
`304`	`304`	`result.messages`
`305`	`305`	`.filter((msg) =>`
`306`		`- msg.type === 'dependency' \|\| msg.type === 'dir-dependency' ? msg : ''`
	`306`	`+ msg.type === 'dependency' \|\| msg.type === 'dir-dependency' ? msg : '',`
`307`	`307`	`)`
`308`	`308`	`.map(depGraph.add)`
`309`	`309`	`.forEach((dependency) => {`
`310`	`310`	`if (dependency.type === 'dir-dependency') {`
`311`	`311`	`messages.push(`
`312`	`312`	`dependency.glob`
`313`	`313`	`? path.join(dependency.dir, dependency.glob)`
`314`		`- : dependency.dir`
	`314`	`+ : dependency.dir,`
`315`	`315`	`)`
`316`	`316`	`} else {`
`317`	`317`	`messages.push(dependency.file)`
Original file line number	Diff line number	Diff line change
`@@ -21,11 +21,11 @@ test('--base --dir works', async (t) => {`
`21`	`21`
`22`	`22`	`t.is(`
`23`	`23`	`await read(path.join(dir, 'level-1/level-2/a.css')),`
`24`		`- await read('test/fixtures/base/level-1/level-2/a.css')`
	`24`	`+ await read('test/fixtures/base/level-1/level-2/a.css'),`
`25`	`25`	`)`
`26`	`26`
`27`	`27`	`t.is(`
`28`	`28`	`await read(path.join(dir, 'level-1/b.css')),`
`29`		`- await read('test/fixtures/base/level-1/b.css')`
	`29`	`+ await read('test/fixtures/base/level-1/b.css'),`
`30`	`30`	`)`
`31`	`31`	`})`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ test('--map && writing to stdout', (t) => {`
`22`	`22`	`t.is(code, 1, 'expected non-zero error code')`
`23`	`23`	`t.regex(`
`24`	`24`	`error.toString(),`
`25`		`- /Output Error: Cannot output external sourcemaps when writing to STDOUT/`
	`25`	`+ /Output Error: Cannot output external sourcemaps when writing to STDOUT/,`
`26`	`26`	`)`
`27`	`27`	`})`
`28`	`28`	`})`
`@@ -33,9 +33,9 @@ test('plugin not found', (t) => {`
`33`	`33`	`t.is(code, 1, 'expected non-zero error code')`
`34`	`34`	`t.regex(`
`35`	`35`	`error.toString(),`
`36`		`- /Plugin Error: Cannot find package 'postcss-plugin'/`
	`36`	`+ /Plugin Error: Cannot find package 'postcss-plugin'/,`
`37`	`37`	`)`
`38`		`- }`
	`38`	`+ },`
`39`	`39`	`)`
`40`	`40`	`})`
`41`	`41`
`@@ -58,8 +58,8 @@ test('CssSyntaxError', (t) => {`
`58`	`58`	`t.is(code, 1, 'expected non-zero error code')`
`59`	`59`	`t.regex(`
`60`	`60`	`error.toString(),`
`61`		`- /CssSyntaxError: .*a.css:1:4: Unnecessary curly bracket/`
	`61`	`+ /CssSyntaxError: .*a.css:1:4: Unnecessary curly bracket/,`
`62`	`62`	`)`
`63`		`- }`
	`63`	`+ },`
`64`	`64`	`)`
`65`	`65`	`})`
Original file line number	Diff line number	Diff line change
`@@ -19,14 +19,14 @@ test('works with glob patterns', async (t) => {`
`19`	`19`
`20`	`20`	`t.is(`
`21`	`21`	`await read(path.join(output, 'a.css')),`
`22`		`- await read('test/fixtures/glob/a.css')`
	`22`	`+ await read('test/fixtures/glob/a.css'),`
`23`	`23`	`)`
`24`	`24`	`t.is(`
`25`	`25`	`await read(path.join(output, 'b.css')),`
`26`		`- await read('test/fixtures/glob/b.css')`
	`26`	`+ await read('test/fixtures/glob/b.css'),`
`27`	`27`	`)`
`28`	`28`	`t.is(`
`29`	`29`	`await read(path.join(output, 's.css')),`
`30`		`- await read('test/fixtures/glob/s.css')`
	`30`	`+ await read('test/fixtures/glob/s.css'),`
`31`	`31`	`)`
`32`	`32`	`})`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ export default function (args, cwd) {`
`13`	`13`	`stdout,`
`14`	`14`	`stderr,`
`15`	`15`	`})`
`16`		`- }`
	`16`	`+ },`
`17`	`17`	`)`
`18`	`18`	`})`
`19`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,6 @@ import fs from 'fs-extra'`
`2`	`2`
`3`	`3`	`export default function (path) {`
`4`	`4`	`return fs.readFile(path, 'utf8').then(`
`5`		`- (content) => content.replace(/\r\n/g, '\n') // normalize line endings on Windows`
	`5`	`+ (content) => content.replace(/\r\n/g, '\n'), // normalize line endings on Windows`
`6`	`6`	`)`
`7`	`7`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,6 @@ test('--version', async (t) => {`
`17`	`17`
`18`	`18`	`t.truthy(`
`19`	`19`	`version.stdout.length > 5,`
`20`		`- 'expected --version to output version info'`
	`20`	`+ 'expected --version to output version info',`
`21`	`21`	`)`
`22`	`22`	`})`