custom-property-pattern rule changed for compatibility with WP naming

[cawa-93]

Type of change

MAJOR: breaking change
fix #2538

[kellymears]

We will want to make this change here, because it is a WordPress specific rule and should not be applied to everyone using sass with bud.js.

https://github.com/roots/bud/tree/main/sources/%40roots/bud-preset-wordpress/config/stylelint.cjs#L1-L8

[cawa-93]

@kellymears moved

[kellymears]

Okay, I ran some tests on this and it's looking good.

Can you add the following test to @sources/@roots/bud-preset-wordpress/test/stylelint-config.test.ts?

import * as stylelint from 'stylelint'
import {describe, expect, it} from 'vitest'

describe(`@roots/bud-preset-wordpress/stylelint`, () => {
  it(`should error without rule`, async () => {
    const result = await stylelint.lint({
      code: `
        .foo {
          color: var(--wp--preset--color--primary);
        }
      `,
      config: {
        extends: [
          `@roots/bud-sass/stylelint-config`,
        ],
      },
    })

    expect(result.errored).toBe(true)
  })

  it(`should not error with rule`, async () => {
    const result = await stylelint.lint({
      code: `
        .foo {
          color: var(--wp--preset--color--primary);
        }
      `,
      config: {
        extends: [
          `@roots/bud-sass/stylelint-config`,
          `@roots/bud-preset-wordpress/stylelint`,
        ],
      },
    })

    expect(result.errored).toBe(false)
  })
})

[kellymears]

thank you very much!