From 1e96ca9204c0177631108149701a4b7e4713bf20 Mon Sep 17 00:00:00 2001
From: Sabina Talipova <sabina.talipova@silverstripe.com>
Date: Mon, 22 Apr 2024 14:54:48 +1200
Subject: [PATCH] MNT Replace sass-lint with stylelint

---
 .sass-lint.yml  | 97 -------------------------------------------------
 .stylelintrc.js |  1 +
 package.json    |  6 +--
 3 files changed, 4 insertions(+), 100 deletions(-)
 delete mode 100644 .sass-lint.yml
 create mode 100644 .stylelintrc.js

diff --git a/.sass-lint.yml b/.sass-lint.yml
deleted file mode 100644
index 3c67d921..00000000
--- a/.sass-lint.yml
+++ /dev/null
@@ -1,97 +0,0 @@
-# sass-lint config to match the AirBNB style guide
-# See silverstripe-admin
-files:
-  include: 'client/src/**/*.scss'
-options:
-  formatter: stylish
-  merge-default-rules: false
-rules:
-  # Warnings
-  # Things that require actual refactoring are marked as warnings
-  class-name-format:
-    - 1
-    - convention: hyphenatedbem
-  placeholder-name-format:
-    - 1
-    - convention: hyphenatedlowercase
-  nesting-depth:
-    - 1
-    - max-depth: 3
-  no-ids: 1
-  no-important: 1
-  no-misspelled-properties:
-    - 1
-    - extra-properties:
-       - "-moz-border-radius-topleft"
-       - "-moz-border-radius-topright"
-       - "-moz-border-radius-bottomleft"
-       - "-moz-border-radius-bottomright"
-  variable-name-format:
-    - 1
-    - allow-leading-underscore: true
-      convention: hyphenatedlowercase
-  no-extends: 1
-
-  # Warnings: these things are preferential rather than mandatory
-  no-css-comments: 1
-
-  # Errors
-  # Things that can be easily fixed are marked as errors
-  indentation:
-    - 2
-    - size: 2
-  final-newline:
-    - 2
-    - include: true
-  no-trailing-whitespace: 2
-  border-zero:
-    - 2
-    - convention: '0'
-  brace-style:
-    - 2
-    - allow-single-line: true
-  clean-import-paths:
-    - 2
-    - filename-extension: false
-      leading-underscore: false
-  no-debug: 2
-  no-empty-rulesets: 2
-  no-invalid-hex: 2
-  no-mergeable-selectors: 2
-  # no-qualifying-elements:
-  #   - 1
-  #   - allow-element-with-attribute: false
-  #     allow-element-with-class: false
-  #     allow-element-with-id: false
-  no-trailing-zero: 2
-  no-url-protocols: 2
-  quotes:
-    - 2
-    - style: double
-  space-after-bang:
-    - 2
-    - include: false
-  space-after-colon:
-    - 2
-    - include: true
-  space-after-comma:
-    - 2
-    - include: true
-  space-before-bang:
-    - 2
-    - include: true
-  space-before-brace:
-    - 2
-    - include: true
-  space-before-colon: 2
-  space-between-parens:
-    - 2
-    - include: false
-  trailing-semicolon: 2
-  url-quotes: 2
-  zero-unit: 2
-  single-line-per-selector: 2
-  one-declaration-per-line: 2
-  empty-line-between-blocks:
-    - 2
-    - ignore-single-line-rulesets: true
\ No newline at end of file
diff --git a/.stylelintrc.js b/.stylelintrc.js
new file mode 100644
index 00000000..ef7353b7
--- /dev/null
+++ b/.stylelintrc.js
@@ -0,0 +1 @@
+module.exports = require('@silverstripe/eslint-config/.stylelintrc');
diff --git a/package.json b/package.json
index 7e854d24..c0fe951a 100644
--- a/package.json
+++ b/package.json
@@ -6,7 +6,7 @@
     "dev": "NODE_ENV=development webpack --progress",
     "watch": "yarn && NODE_ENV=development webpack --watch --progress",
     "css": "WEBPACK_CHILD=css npm run build",
-    "lint": "eslint client/src; sass-lint -v"
+    "lint": "eslint client/src; stylelint client/src/**/*.scss"
   },
   "repository": {
     "type": "git",
@@ -25,8 +25,8 @@
   "homepage": "https://github.com/symbiote/silverstripe-advancedworkflow#readme",
   "dependencies": {},
   "devDependencies": {
-    "@silverstripe/eslint-config": "^1.1.0",
-    "@silverstripe/webpack-config": "^2.0.0",
+    "@silverstripe/eslint-config": "^1.2.1",
+    "@silverstripe/webpack-config": "^2.1.0",
     "webpack": "^5.74.0",
     "webpack-cli": "^5.0.0"
   },