Skip to content

Conversation

@JammingBen
Copy link
Contributor

With the recent removal of our sass files, these are not needed anymore.

supersedes #1254

With the recent removal of our sass files, these are not needed anymore.
@JammingBen JammingBen self-assigned this Sep 24, 2025
Copilot AI review requested due to automatic review settings September 24, 2025 06:03
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 Sass-related dependencies from the design system and extension SDK packages following the removal of Sass files from the codebase.

  • Removes Sass peer dependency from extension-sdk package
  • Removes Sass development dependencies (sass, sass-loader, sass-resources-loader) from design-system package

Reviewed Changes

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

File Description
packages/extension-sdk/package.json Removes sass peer dependency
packages/design-system/package.json Removes sass, sass-loader, and sass-resources-loader dev dependencies
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@JammingBen JammingBen requested a review from kulmann September 24, 2025 06:27
@kulmann kulmann merged commit 2925800 into main Sep 24, 2025
29 checks passed
@kulmann kulmann deleted the chore/remove-sass-dependencies branch September 24, 2025 10:01
@openclouders openclouders mentioned this pull request Sep 24, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants