ci: run gulp bundle on Node 20

.github/workflows/main.yml

@@ -29,12 +29,14 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Check out repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
+      # Node 17+ dropped PostCSS 8.1's "./" exports fallback, which broke
+      # gulp-stylelint 13 / postcss-syntax. CSS lint now uses stylelint 14.
       - name: Set up Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
-          node-version: 20
+          node-version-file: .nvmrc
           cache: npm
 
       - name: Install dependencies
@@ -44,7 +46,7 @@ jobs:
         run: npx gulp bundle
 
       - name: Upload bundle artifact
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v6
         with:
           name: ui-bundle
           path: build/ui-bundle.zip

.nvmrc

@@ -1 +1 @@
-10
+20

.stylelintrc

@@ -3,5 +3,19 @@
   "rules": {
     "comment-empty-line-before": null,
     "no-descending-specificity": null,
+    "value-keyword-case": ["lower", { "ignoreKeywords": ["Arial", "currentColor"] }],
+    "custom-property-pattern": null,
+    "selector-class-pattern": null,
+    "keyframes-name-pattern": null,
+    "function-url-quotes": null,
+    "font-family-name-quotes": null,
+    "string-quotes": null,
+    "color-function-notation": null,
+    "alpha-value-notation": null,
+    "value-no-vendor-prefix": null,
+    "property-no-vendor-prefix": null,
+    "selector-attribute-quotes": null,
+    "shorthand-property-no-redundant-values": null,
+    "max-line-length": null
   }
 }

gulp.d/tasks/lint-css.js

@@ -1,10 +1,13 @@
 'use strict'
 
-const stylelint = require('gulp-stylelint')
-const vfs = require('vinyl-fs')
+const stylelint = require('stylelint')
 
-module.exports = (files) => (done) =>
-  vfs
-    .src(files)
-    .pipe(stylelint({ reporters: [{ formatter: 'string', console: true }], failAfterError: true }))
-    .on('error', done)
+module.exports = (files) => () =>
+  stylelint.lint({ files, formatter: 'string' }).then((result) => {
+    if (result.output) process.stdout.write(result.output)
+    if (result.errored) {
+      const error = new Error('Failed with stylelint errors')
+      error.showStack = false
+      throw error
+    }
+  })