Add trailing commas; lint on CI (#553)

* Fix ci to actually lint

* Add trailing commas

Author: RyanZim

.github/workflows/ci.yaml

@@ -18,4 +18,4 @@ jobs:
       with:
         node-version: ${{ matrix.node-version }}
     - run: npm install
-    - run: npm test
+    - run: npm run ci

index.js

@@ -55,7 +55,7 @@ function AtImport(options) {
         state,
         [],
         [],
-        postcss
+        postcss,
       )
 
       applyRaws(bundle)

lib/apply-conditions.js

@@ -15,7 +15,7 @@ module.exports = function applyConditions(bundle, atRule) {
     if (stmt.type === "import") {
       stmt.node.params = base64EncodedConditionalImport(
         stmt.fullUri,
-        stmt.conditions
+        stmt.conditions,
       )
       return
     }

lib/base64-encoded-import.js

@@ -13,16 +13,16 @@ module.exports = function base64EncodedConditionalImport(prelude, conditions) {
   let params = `${prelude} ${formatImportPrelude(
     first.layer,
     first.media,
-    first.supports
+    first.supports,
   )}`
 
   for (const condition of conditions) {
     params = `'data:text/css;base64,${Buffer.from(`@import ${params}`).toString(
-      "base64"
+      "base64",
     )}' ${formatImportPrelude(
       condition.layer,
       condition.media,
-      condition.supports
+      condition.supports,
     )}`
   }
 

lib/parse-statements.js

@@ -106,7 +106,7 @@ function parseImport(result, atRule, conditions, from) {
 
       return result.warn(
         "@import must precede all other statements (besides @charset or empty @layer)",
-        { node: atRule }
+        { node: atRule },
       )
     } while (prev)
   }
@@ -115,7 +115,7 @@ function parseImport(result, atRule, conditions, from) {
     return result.warn(
       "It looks like you didn't end your @import statement correctly. " +
         "Child nodes are attached to it.",
-      { node: atRule }
+      { node: atRule },
     )
   }
 
@@ -195,7 +195,7 @@ function parseImport(result, atRule, conditions, from) {
           `layers must be defined before support conditions in '${atRule.toString()}'`,
           {
             node: atRule,
-          }
+          },
         )
       }
 
@@ -214,7 +214,7 @@ function parseImport(result, atRule, conditions, from) {
           `Multiple support conditions in '${atRule.toString()}'`,
           {
             node: atRule,
-          }
+          },
         )
       }
 

lib/parse-styles.js

@@ -15,7 +15,7 @@ async function parseStyles(
   state,
   conditions,
   from,
-  postcss
+  postcss,
 ) {
   const statements = parseStatements(result, styles, conditions, from)
 
@@ -45,7 +45,7 @@ async function parseStyles(
       throw stmt.node.error(
         `Incompatible @charset statements:
   ${stmt.node.params} specified in ${stmt.node.source.input.file}
-  ${charset.node.params} specified in ${charset.node.source.input.file}`
+  ${charset.node.params} specified in ${charset.node.source.input.file}`,
       )
     }
   }
@@ -80,14 +80,14 @@ async function resolveImportId(result, stmt, options, state, postcss) {
       stmt.uri,
       options,
       state,
-      postcss
+      postcss,
     )
 
     return
   } else if (dataURL.isValid(stmt.from.slice(-1))) {
     // Data urls can't be used as a base url to resolve imports.
     throw stmt.node.error(
-      `Unable to import '${stmt.uri}' from a stylesheet that is embedded in a data url`
+      `Unable to import '${stmt.uri}' from a stylesheet that is embedded in a data url`,
     )
   }
 
@@ -108,7 +108,7 @@ async function resolveImportId(result, stmt, options, state, postcss) {
       return !path.isAbsolute(file)
         ? resolveId(file, base, options, atRule)
         : file
-    })
+    }),
   )
 
   // Add dependency messages:
@@ -124,7 +124,7 @@ async function resolveImportId(result, stmt, options, state, postcss) {
   const importedContent = await Promise.all(
     resolved.map(file => {
       return loadImportContent(result, stmt, file, options, state, postcss)
-    })
+    }),
   )
 
   // Merge loaded statements
@@ -138,13 +138,13 @@ async function loadImportContent(
   filename,
   options,
   state,
-  postcss
+  postcss,
 ) {
   const atRule = stmt.node
   const { conditions, from } = stmt
   const stmtDuplicateCheckKey = conditions
     .map(condition =>
-      formatImportPrelude(condition.layer, condition.media, condition.supports)
+      formatImportPrelude(condition.layer, condition.media, condition.supports),
     )
     .join(":")
 
@@ -185,7 +185,7 @@ async function loadImportContent(
     content,
     filename,
     options,
-    postcss
+    postcss,
   )
 
   const styles = importedResult.root
@@ -213,7 +213,7 @@ async function loadImportContent(
     state,
     conditions,
     [...from, filename],
-    postcss
+    postcss,
   )
 }
 

lib/process-content.js

@@ -11,7 +11,7 @@ module.exports = function processContent(
   content,
   filename,
   options,
-  postcss
+  postcss,
 ) {
   const { plugins } = options
   const ext = path.extname(filename)

lib/resolve-id.js

@@ -36,7 +36,7 @@ module.exports = function resolveId(id, base, options, node) {
         `Failed to find '${id}'
   in [
     ${paths.join(",\n    ")}
-  ]`
+  ]`,
       )
     })
 }

test/custom-resolve.js

@@ -23,10 +23,10 @@ test.serial(
   {
     resolve: () => {
       return Promise.resolve(
-        path.resolve("test/fixtures/imports/custom-resolve-1.css")
+        path.resolve("test/fixtures/imports/custom-resolve-1.css"),
       )
     },
-  }
+  },
 )
 
 test.serial(
@@ -41,7 +41,7 @@ test.serial(
         path.resolve("test/fixtures/imports/custom-resolve-1.css"),
       ]
     },
-  }
+  },
 )
 
 test.serial(
@@ -56,7 +56,7 @@ test.serial(
         path.resolve("test/fixtures/imports/custom-resolve-1.css"),
       ])
     },
-  }
+  },
 )
 
 test("should apply default resolver when custom doesn't return an absolute path", t => {
@@ -68,7 +68,7 @@ test("should apply default resolver when custom doesn't return an absolute path"
           t.is(p, path.resolve("test/fixtures/imports", "bar.css"))
           return "/* comment */"
         },
-      })
+      }),
     )
     .process(`@import "foo.css";`, {
       from: "test/fixtures/custom-resolve-file",

test/custom-syntax-parser.js

@@ -16,7 +16,7 @@ test(
   checkFixture,
   "scss-parser",
   null,
-  { parser: scss }
+  { parser: scss },
 )
 
 test(".css importing .sss should work", checkFixture, "import-sss")
@@ -26,27 +26,27 @@ test(
   checkFixture,
   { name: "sugar", ext: ".sss" },
   null,
-  { parser: sugarss }
+  { parser: sugarss },
 )
 
 test(
   ".sss importing .css should work",
   checkFixture,
   { name: "sugar-import-css", ext: ".sss" },
   null,
-  { parser: sugarss }
+  { parser: sugarss },
 )
 
 test(
   ".css importing .sss importing .css should work",
   checkFixture,
-  "import-sss-css"
+  "import-sss-css",
 )
 
 test(
   ".sss importing .css importing .sss should work",
   checkFixture,
   { name: "import-css-sss", ext: ".sss" },
   null,
-  { parser: sugarss }
+  { parser: sugarss },
 )