Repository: parcel-bundler/parcel (js)

Issue:
Sass not SASS, rename
<!---
Thanks for filing an issue 😄 ! Before you submit, please read the following:

Search open/closed issues before submitting since someone might have asked the same thing before!
-->

# 🐛 bug report

<!--- Provide a general summary of the issue here -->

## 🎛 Configuration (.babelrc, package.json, cli command)

n/a

## 🤔 Expected Behavior

Files/strings named `SassAsset.js ` and `should create a valid sourcemap for a Sass asset`

## 😯 Current Behavior

Files/strings named `SASSAsset.js `and `should create a valid sourcemap for a SASS asset`

## 💁 Possible Solution

Rename it. Maybe rename LESS to Less as well, although Less is sometimes stylized as LESS whereas Sass is not.

## 🔦 Context

Nothing is "broken", but Sass is spelled wrong. There's even a website, [Sass not SASS](http://sassnotsass.com/), dedicated to this.



Pull request:
rename SASS → Sass
Fixes #5147 

Some places this was already correct, but not all. Now it's consistently "Sass".

Patch:
diff --git a/CHANGELOG.md b/CHANGELOG.md
index cc31ee377..05e80d0e8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -27,7 +27,7 @@ and Parcel adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ### Added
 
-- CSS/SASS/LESS sourcemaps [Details](https://github.com/parcel-bundler/parcel/pull/2489)
+- CSS/Sass/LESS sourcemaps [Details](https://github.com/parcel-bundler/parcel/pull/2489)
 - Add Markdown support [Details](https://github.com/parcel-bundler/parcel/pull/2538)
 - Unhandled HMR updates should cause a page reload [Details](https://github.com/parcel-bundler/parcel/pull/2676)
 - Enables jsx plugin in case jsx syntax is used in js files [Details](https://github.com/parcel-bundler/parcel/pull/2530)
diff --git a/packages/core/parcel-bundler/src/Parser.js b/packages/core/parcel-bundler/src/Parser.js
index 82cf98488..688ef9f6c 100644
--- a/packages/core/parcel-bundler/src/Parser.js
+++ b/packages/core/parcel-bundler/src/Parser.js
@@ -37,8 +37,8 @@ class Parser {
     this.registerExtension('styl', './assets/StylusAsset');
     this.registerExtension('stylus', './assets/StylusAsset');
     this.registerExtension('less', './assets/LESSAsset');
-    this.registerExtension('sass', './assets/SASSAsset');
-    this.registerExtension('scss', './assets/SASSAsset');
+    this.registerExtension('sass', './assets/SassAsset');
+    this.registerExtension('scss', './assets/SassAsset');
 
     this.registerExtension('html', './assets/HTMLAsset');
     this.registerExtension('htm', './assets/HTMLAsset');
diff --git a/packages/core/parcel-bundler/src/assets/SASSAsset.js b/packages/core/parcel-bundler/src/assets/SASSAsset.js
index 153b99c59..eec95a781 100644
--- a/packages/core/parcel-bundler/src/assets/SASSAsset.js
+++ b/packages/core/parcel-bundler/src/assets/SASSAsset.js
@@ -6,7 +6,7 @@ const os = require('os');
 const Resolver = require('../Resolver');
 const parseCSSImport = require('../utils/parseCSSImport');
 
-class SASSAsset extends Asset {
+class SassAsset extends Asset {
   constructor(name, options) {
     super(name, options);
     this.type = 'css';
@@ -95,7 +95,7 @@ class SASSAsset extends Asset {
   }
 }
 
-module.exports = SASSAsset;
+module.exports = SassAsset;
 
 async function getSassRuntime(searchPath) {
   try {


Test patch:
diff --git a/packages/core/integration-tests/data/lastTestRun.json b/packages/core/integration-tests/data/lastTestRun.json
index 3f4721f66..7a51a988d 100644
--- a/packages/core/integration-tests/data/lastTestRun.json
+++ b/packages/core/integration-tests/data/lastTestRun.json
@@ -2559,13 +2559,13 @@
     {
       "fileName": "sourcemaps",
       "link": "https://github.com/parcel-bundler/parcel/tree/v2/packages/core/integration-tests/test/sourcemaps.js",
-      "title": "should create a valid sourcemap for a SASS asset",
+      "title": "should create a valid sourcemap for a Sass asset",
       "status": "pending"
     },
     {
       "fileName": "sourcemaps",
       "link": "https://github.com/parcel-bundler/parcel/tree/v2/packages/core/integration-tests/test/sourcemaps.js",
-      "title": "should create a valid sourcemap when for a CSS asset importing SASS",
+      "title": "should create a valid sourcemap when for a CSS asset importing Sass",
       "status": "pending"
     },
     {
diff --git a/packages/core/integration-tests/test/sourcemaps.js b/packages/core/integration-tests/test/sourcemaps.js
index 7ed51e241..6b4f7fd78 100644
--- a/packages/core/integration-tests/test/sourcemaps.js
+++ b/packages/core/integration-tests/test/sourcemaps.js
@@ -683,7 +683,7 @@ describe('sourcemaps', function() {
     await test(true);
   });
 
-  it('should create a valid sourcemap for a SASS asset', async function() {
+  it('should create a valid sourcemap for a Sass asset', async function() {
     async function test(minify) {
       let inputFilePath = path.join(
         __dirname,
@@ -738,7 +738,7 @@ describe('sourcemaps', function() {
     await test(true);
   });
 
-  it('should create a valid sourcemap when for a CSS asset importing SASS', async function() {
+  it('should create a valid sourcemap when for a CSS asset importing Sass', async function() {
     async function test(minify) {
       let inputFilePath = path.join(
         __dirname,