Upgrading to stylelint 15.10.2 (#356)

* Upgrading to stylint 15.10.2

Co-authored-by: Katie Langerman <langermank@users.noreply.github.com>

* Change to npm

* Fixing tests

* Fix lint

* Updating rules to match dotcom

* Create lazy-bikes-roll.md

---------

Co-authored-by: Katie Langerman <langermank@users.noreply.github.com>

.changeset/lazy-bikes-roll.md

@@ -0,0 +1,5 @@
+---
+"@primer/stylelint-config": minor
+---
+
+Upgrading to stylelint 15.10.2

.github/workflows/ci.yml

@@ -12,14 +12,14 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 16
-      - run: yarn
-      - run: yarn test
+      - run: npm ci
+      - run: npm run test
   lint:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
           node-version: 16
-      - run: yarn
-      - run: yarn lint
+      - run: npm ci
+      - run: npm run lint

.github/workflows/release.yml

@@ -23,15 +23,15 @@ jobs:
           node-version: 16
 
       - name: Install dependencies
-        run: yarn
+        run: npm ci
 
       - name: Create release pull request or publish to npm
         id: changesets
         uses: changesets/action@master
         with:
           title: Release Tracking
           # This expects you to have a script called release which does a build for your packages and calls changeset publish
-          publish: yarn release
+          publish: npm run release
         env:
           GITHUB_TOKEN: ${{ secrets.GPR_AUTH_TOKEN_SHARED }}
           NPM_TOKEN: ${{ secrets.NPM_AUTH_TOKEN_SHARED }}

.github/workflows/release_canary.yml

@@ -25,7 +25,7 @@ jobs:
           node-version: 16
 
       - name: Install dependencies
-        run: yarn
+        run: npm ci
 
       - name: Create .npmrc
         run: |
@@ -40,8 +40,8 @@ jobs:
           echo "$( jq '.version = "0.0.0"' package.json )" > package.json
           echo -e "---\n'@primer/stylelint-config': patch\n---\n\nFake entry to force publishing" > .changeset/force-snapshot-release.md
           rm -f .changeset/pre.json
-          yarn changeset version --snapshot
-          yarn changeset publish --tag canary
+          npx changeset version --snapshot
+          npx changeset publish --tag canary
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 

.github/workflows/release_candidate.yml

@@ -23,7 +23,7 @@ jobs:
           node-version: 16
 
       - name: Install dependencies
-        run: yarn
+        run: npm ci
 
       - name: Create .npmrc
         run: |
@@ -37,7 +37,7 @@ jobs:
         run: |
           version=$(jq -r .version package.json)
           echo "$( jq ".version = \"$(echo $version)-rc.$(git rev-parse --short HEAD)\"" package.json )" > package.json
-          yarn publish --tag next
+          npm publish --tag next
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 

.gitignore

@@ -1,4 +1,4 @@
 *.log
 coverage/
 node_modules/
-package-lock.json
+yarn.lock

.npmrc

@@ -1,2 +1 @@
 git-tag-version=false
-package-lock=false

__tests__/index.js

@@ -29,13 +29,14 @@ describe('stylelint-config', () => {
         .foo {
           width: 10px;
           top: .2em;
+          max-width: initial;
         }
       `).then(data => {
       expect(data).toHaveErrored()
       expect(data).toHaveWarningsLength(2)
       expect(data).toHaveWarnings([
         'Expected "top" to come before "width" (order/properties-order)',
-        'Expected a leading zero (number-leading-zero)',
+        'Unexpected value "initial" for property "max-width" (declaration-property-value-disallowed-list)',
       ])
     })
   })

index.js

@@ -36,7 +36,6 @@ module.exports = {
     'declaration-block-no-duplicate-properties': [true, {ignore: ['consecutive-duplicates']}],
     'declaration-block-no-redundant-longhand-properties': null,
     'declaration-block-no-shorthand-property-overrides': true,
-    'declaration-block-semicolon-newline-after': 'always',
     'declaration-property-value-disallowed-list': {
       '/^transition/': ['/all/'],
       '/^background/': ['http:', 'https:'],
@@ -47,13 +46,11 @@ module.exports = {
     'function-linear-gradient-no-nonstandard-direction': true,
     'function-no-unknown': null,
     'keyframes-name-pattern': null,
-    'max-line-length': null,
     'max-nesting-depth': 3,
     'media-feature-name-no-unknown': null,
     'media-feature-name-no-vendor-prefix': null,
     'no-descending-specificity': null,
     'no-duplicate-selectors': true,
-    'no-extra-semicolons': true,
     'no-invalid-position-at-import-rule': [true, {ignoreAtRules: ['use']}],
     'number-max-precision': null,
     'order/properties-order': propertyOrder,
@@ -94,8 +91,13 @@ module.exports = {
     'selector-no-qualifying-type': true,
     'selector-pseudo-element-no-unknown': true,
     'string-no-newline': true,
-    'string-quotes': 'single',
     'unit-no-unknown': true,
     'value-keyword-case': null,
+    'selector-not-notation': null,
+    'import-notation': ['string'],
+    'annotation-no-unknown': null,
+    'keyframe-selector-notation': ['percentage-unless-within-keyword-only-block'],
+    'media-query-no-invalid': null,
+    'media-feature-range-notation': ['prefix'],
   },
 }