Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove outdated Ruby-Sass and Node-Sass dependencies #3952

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Aic-02mz
Copy link

Remove outdated Ruby-Sass and Node-Sass dependencies

This pull request proposes the removal of outdated dependencies, specifically Ruby-Sass and Node-Sass, to improve the overall maintainability and performance of the repository. Both dependencies are no longer actively supported and they are both out of date

Key Changes:

img2

Removed Ruby-Sass and Node-Sass files:

Removed redundant files for the Sass test folder.

Streamlined the test directory:

The test directory now only contains Dart-Sass, reducing clutter and complexity.

Resolved Node-Sass syntax issues:

Adjustments have been made to eliminate deprecated syntax.

Rationale:

Both Ruby-Sass and Node-Sass are no longer actively supported.

The repository now fully aligns with modern standards by using Dart-Sass exclusively.

This change reduces potential maintenance in the future by removing outdated tools.

Testing:

This PR primarily involves the clean-up and reorganization of files. No testing was performed, as no new functionality or features were introduced.

Before and After:

The directory now focuses solely on Dart-Sass, which is the preferred and modern standard.

Changelog:

Not needed, as this is a clean-up effort.

Additional Notes:

This aligns the repository with current best practices by removing unsupported tools and ensuring a cleaner structure.

…d maintainability, as both are outdated and no longer actively supported.

- Node-Sass syntax issues have been resolved.
- Cleaned up test folder structure for better organization.

These changes align the repository with best practices of keeping everything up to date.
@Aic-02mz Aic-02mz mentioned this pull request Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants