Remove sourcemap comments from Sass sources (#1860)

* Remove sourcemap comments from Sass sources * add changelog * update version to cut a release and update sass_api changelog and pubspec
sass · Jan 5, 2023 · aceb1e8 · aceb1e8
1 parent 441be80
commit aceb1e8
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,8 @@
+## 1.57.2
+
+* Remove sourcemap comments from Sass sources. The generated sourcemap comment
+  for the compiled CSS output remains unaffected.
+
 ## 1.57.1
 
 * No user-visible changes.

diff --git a/lib/src/visitor/serialize.dart b/lib/src/visitor/serialize.dart
@@ -179,6 +179,9 @@ class _SerializeVisitor
       // Preserve comments that start with `/*!`.
       if (_isCompressed && !node.isPreserved) return;
 
+      // Ignore sourceMappingURL and sourceURL comments.
+      if (node.text.startsWith(RegExp(r"/\*# source(Mapping)?URL="))) return;
+
       var minimumIndentation = _minimumIndentation(node.text);
       assert(minimumIndentation != -1);
       if (minimumIndentation == null) {

diff --git a/pkg/sass_api/CHANGELOG.md b/pkg/sass_api/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 4.2.2
+
+* No user-visible changes.
+
 ## 4.2.1
 
 * No user-visible changes.

diff --git a/pkg/sass_api/pubspec.yaml b/pkg/sass_api/pubspec.yaml
@@ -2,15 +2,15 @@ name: sass_api
 # Note: Every time we add a new Sass AST node, we need to bump the *major*
 # version because it's a breaking change for anyone who's implementing the
 # visitor interface(s).
-version: 4.2.1
+version: 4.2.2
 description: Additional APIs for Dart Sass.
 homepage: https://github.com/sass/dart-sass
 
 environment:
   sdk: ">=2.17.0 <3.0.0"
 
 dependencies:
-  sass: 1.57.1
+  sass: 1.57.2
 
 dev_dependencies:
   dartdoc: ^5.0.0

diff --git a/pubspec.yaml b/pubspec.yaml
@@ -1,5 +1,5 @@
 name: sass
-version: 1.57.1
+version: 1.57.2
 description: A Sass implementation in Dart.
 homepage: https://github.com/sass/dart-sass