Skip to content

Conversation

@pharret31
Copy link
Contributor

No description provided.

@pharret31 pharret31 self-assigned this Nov 26, 2025
@pharret31 pharret31 added the 25_2 label Nov 26, 2025
@pharret31 pharret31 marked this pull request as ready for review November 26, 2025 12:24
Copilot AI review requested due to automatic review settings November 26, 2025 12:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the dependency on the stylelint-devextreme plugin (version 0.0.6) from the DevExtreme monorepo. The plugin provided two custom stylelint rules that enforced DevExtreme-specific SCSS conventions:

  1. devextreme/at-use-no-public-components - enforced restrictions on using public components via @use statements
  2. devextreme/no-division-slash - enforced restrictions on the deprecated SCSS division slash operator

Key Changes

  • Removed stylelint-devextreme package dependency from the devextreme package and pnpm lockfile
  • Removed the plugin from the stylelint configuration for the devextreme-scss package
  • Removed the two custom rule configurations from the stylelint config

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
pnpm-lock.yaml Removed stylelint-devextreme@0.0.6 package from importers and snapshots sections
packages/devextreme/package.json Removed stylelint-devextreme from devDependencies
packages/devextreme-scss/.stylelintrc.json Removed plugin from plugins array and deleted custom rule configurations (devextreme/at-use-no-public-components and devextreme/no-division-slash)
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

@pharret31 pharret31 merged commit d31f95e into DevExpress:25_2 Nov 26, 2025
102 of 103 checks passed
@pharret31 pharret31 deleted the 25_2_2897-remove-dependency-on-the-stylelint-devextreme-plugin branch November 26, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants