Skip to content

Conversation

querkmachine
Copy link
Member

@querkmachine querkmachine commented Oct 6, 2025

Closes #6242 and #6243. Part of #2637.

Warning

We're treating the removal of support for Ruby Sass and LibSass as a breaking change. This PR should not be merged until we're preparing to release Frontend v6.0.0.

Prior to merging, the branch protection rules for main will need to be updated to no longer require the Ruby Sass and LibSass tests to have been ran.

Changes

  • Removes CI tests for Ruby Sass and LibSass (including node-sass).
  • Removes deprecation warning shown to LibSass users.

@querkmachine querkmachine self-assigned this Oct 6, 2025
@querkmachine querkmachine changed the title Remove Ruby Sass and LibSass Remove support for Ruby Sass and LibSass Oct 6, 2025
Copy link

github-actions bot commented Oct 6, 2025

Other changes to npm package

diff --git a/packages/govuk-frontend/dist/govuk/settings/_index.scss b/packages/govuk-frontend/dist/govuk/settings/_index.scss
index 9cbf2779d..e46fb4637 100644
--- a/packages/govuk-frontend/dist/govuk/settings/_index.scss
+++ b/packages/govuk-frontend/dist/govuk/settings/_index.scss
@@ -20,17 +20,4 @@
 
 @import "links";
 
-// LibSass evaluates this as a single unquoted string, equivalent to
-// "true == false", whereas Dart Sass evaluates it as "true" == "false".
-// Because a string in Sass is truthy, LibSass will output the warning; Dart
-// Sass will not.
-@if #{true} == #{false} {
-  @include _warning(
-    "libsass",
-    "It looks like you may be using LibSass to compile your Sass. LibSass is " +
-      "deprecated and will not be supported by the next major version of " +
-      "GOV.UK Frontend. See https://sass-lang.com/libsass/ for more information."
-  );
-}
-
 /*# sourceMappingURL=_index.scss.map */

Action run for 76a08b7

Copy link

github-actions bot commented Oct 6, 2025

📋 Stats

No changes to any distributed file sizes!


Action run for 76a08b7

Copy link
Member

@romaricpascal romaricpascal left a comment

Choose a reason for hiding this comment

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

Looks good to merge.

On top of main we'll need to change the protection rules of our other long-lived branches (feature/* and support/* branches).

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.

Remove the check for compilation using LibSass from govuk/settings/_index.scss Stop testing Sass compilation against Ruby Sass and LibSass compilers

3 participants