Fail dist build when for postcss warnings (#2180)

* Fail dist build when for postcss warnings * Create rotten-phones-ring.md Co-authored-by: Jon Rohan <yes@jonrohan.codes>
primer · Jul 29, 2022 · c71e7df · c71e7df
1 parent 370c8a2
commit c71e7df
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/.changeset/rotten-phones-ring.md b/.changeset/rotten-phones-ring.md
@@ -0,0 +1,5 @@
+---
+"@primer/css": patch
+---
+
+Fail dist build when for postcss warnings
diff --git a/script/dist.js b/script/dist.js
@@ -51,6 +51,15 @@ async function dist() {
       const scss = await readFile(from, encoding)
       meta.imports = getExternalImports(scss, path).map(getPathName)
       const result = await compiler(scss, {from, to})
+      const warnings = result.warnings()
+
+      // We don't want to release changes that cause warnings with postcss. Fail the dist build if any warnings are detected.
+      if (warnings.length) {
+        for (const warning of warnings) {
+          console.warn(warning.toString())
+        }
+        throw new Error(`Warnings while compiling ${from}.  See output above.`)
+      }
 
       await Promise.all([
         writeFile(to, result.css, encoding),