ban new references to kleur/colors

ampproject · May 5, 2021 · 071b268 · 071b268
1 parent 54b60a1
commit 071b268
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/build-system/common/colors.js b/build-system/common/colors.js
@@ -23,7 +23,7 @@ module.exports = Object.entries(colors).reduce((map, [key, formatter]) => {
   }
   map[key] = function (input) {
     if (!input) {
-      return '';
+      return `${input}`;
     }
 
     if (input instanceof Array) {

diff --git a/build-system/test-configs/dep-check-config.js b/build-system/test-configs/dep-check-config.js
@@ -553,4 +553,10 @@ exports.rules = [
       /** DO NOT ADD TO ALLOWLIST */
     ],
   },
+  // Rules for build-system
+  {
+    filesMatching: 'build-system/**/*.js',
+    mustNotDependOn: 'kleur/colors',
+    allowlist: ['build-system/common/colors.js'],
+  },
 ];