use === consistently instead of sometimes == (#313)

Author: bartveneman

src/index.js

@@ -150,7 +150,7 @@ const analyze = (css) => {
 
           node.block.children.forEach(descriptor => {
             // Ignore 'Raw' nodes in case of CSS syntax errors
-            if (descriptor.type == 'Declaration') {
+            if (descriptor.type === 'Declaration') {
               descriptors[descriptor.property] = stringifyNode(descriptor.value)
             }
           })

src/selectors/utils.js

@@ -56,18 +56,18 @@ export function isAccessibility(selector) {
   let isA11y = false
 
   walk(selector, function (node) {
-    if (node.type == 'AttributeSelector') {
+    if (node.type === 'AttributeSelector') {
       if (strEquals('role', node.name.name) || startsWith('aria-', node.name.name)) {
         isA11y = true
         return this.break
       }
     }
     // Test for [aria-] or [role] inside :is()/:where() and friends
-    else if (node.type == 'PseudoClassSelector') {
+    else if (node.type === 'PseudoClassSelector') {
       if (isPseudoFunction(node.name)) {
         const list = analyzeList(node, isAccessibility)
 
-        if (list.some(b => b == true)) {
+        if (list.some(b => b === true)) {
           isA11y = true
           return this.skip
         }
@@ -90,23 +90,23 @@ export function getComplexity(selector) {
   let isPrefixed = false
 
   walk(selector, function (node) {
-    if (node.type == 'Selector' || node.type == 'Nth') return
+    if (node.type === 'Selector' || node.type === 'Nth') return
 
     complexity++
 
-    if (node.type == 'IdSelector'
-      || node.type == 'ClassSelector'
-      || node.type == 'PseudoElementSelector'
-      || node.type == 'TypeSelector'
-      || node.type == 'PseudoClassSelector'
+    if (node.type === 'IdSelector'
+      || node.type === 'ClassSelector'
+      || node.type === 'PseudoElementSelector'
+      || node.type === 'TypeSelector'
+      || node.type === 'PseudoClassSelector'
     ) {
       if (hasVendorPrefix(node.name)) {
         isPrefixed = true
         complexity++
       }
     }
 
-    if (node.type == 'AttributeSelector') {
+    if (node.type === 'AttributeSelector') {
       if (Boolean(node.value)) {
         complexity++
       }
@@ -117,7 +117,7 @@ export function getComplexity(selector) {
       return this.skip
     }
 
-    if (node.type == 'PseudoClassSelector') {
+    if (node.type === 'PseudoClassSelector') {
       if (isPseudoFunction(node.name)) {
         const list = analyzeList(node, getComplexity)
 
@@ -126,7 +126,7 @@ export function getComplexity(selector) {
 
         list.forEach(([c, p]) => {
           complexity += c
-          if (p == true) isPrefixed = true
+          if (p === true) isPrefixed = true
         })
         return this.skip
       }

src/stylesheet/stylesheet.test.js

@@ -167,7 +167,7 @@ Stylesheet('reports embed size correctly when there are duplicates', () => {
     total: 2,
     totalUnique: 1,
     unique: {
-      // .length == 459
+      // .length === 459
       'data:image/svg+xml,%3Csvg%20id%3D%22Layer_1%22%20xmlns%3D%22http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%22%20viewBox%3D%220%200%20195.6%20107.8%22%3E%3Cpath%20fill%3D%22%23B5B5B5%22%20class%3D%22st0%22%20d%3D%22M97.8%20107.8c-2.6%200-5.1-1-7.1-2.9L2.9%2017.1C-1%2013.2-1%206.8%202.9%202.9%206.8-1%2013.2-1%2017.1%202.9l80.7%2080.7%2080.7-80.7c3.9-3.9%2010.2-3.9%2014.1%200%203.9%203.9%203.9%2010.2%200%2014.1l-87.8%2087.8c-1.9%202-4.4%203-7%203z%22%2F%3E%3C%2Fsvg%3E': 2,
     },
     uniquenessRatio: 1 / 2,

src/values/destructure-font-shorthand.js

@@ -46,8 +46,8 @@ export function destructure(value, stringifyNode) {
 		// any node that comes before the '/' is the font-size
 		if (
 			item.next &&
-			item.next.data.type == 'Operator' &&
-			item.next.data.value.charCodeAt(0) == SLASH
+			item.next.data.type === 'Operator' &&
+			item.next.data.value.charCodeAt(0) === SLASH
 		) {
 			font_size = stringifyNode(node)
 			return
@@ -56,8 +56,8 @@ export function destructure(value, stringifyNode) {
 		// any node that comes after '/' is the line-height
 		if (
 			item.prev &&
-			item.prev.data.type == 'Operator' &&
-			item.prev.data.value.charCodeAt(0) == SLASH
+			item.prev.data.type === 'Operator' &&
+			item.prev.data.value.charCodeAt(0) === SLASH
 		) {
 			line_height = stringifyNode(node)
 			return
@@ -66,8 +66,8 @@ export function destructure(value, stringifyNode) {
 		// any node that's followed by ',' is a font-family
 		if (
 			item.next &&
-			item.next.data.type == 'Operator' &&
-			item.next.data.value.charCodeAt(0) == COMMA &&
+			item.next.data.type === 'Operator' &&
+			item.next.data.value.charCodeAt(0) === COMMA &&
 			!font_family[0]
 		) {
 			font_family[0] = node
@@ -84,7 +84,7 @@ export function destructure(value, stringifyNode) {
 			node.type === 'Dimension' &&
 			item.prev &&
 			item.prev.data.type === 'Identifier' &&
-			item.prev.data.name == 'oblique'
+			item.prev.data.name === 'oblique'
 		) {
 			// put in the correct amount of whitespace between `oblique` and `<angle>`
 			font_style +=
@@ -95,12 +95,12 @@ export function destructure(value, stringifyNode) {
 
 		// any node that's a number and not previously caught by line-height or font-size is the font-weight
 		// (oblique <angle> will not be caught here, because that's a Dimension, not a Number)
-		if (node.type == 'Number') {
+		if (node.type === 'Number') {
 			return
 		}
 
 		// last node always ends the font-family
-		if (item.next == null) {
+		if (item.next === null) {
 			font_family[1] = node
 
 			// if, at the last node, we don;t have a size yet, it *must* be the previous node
@@ -113,7 +113,7 @@ export function destructure(value, stringifyNode) {
 		}
 
 		// Any remaining identifiers can be font-size, font-style, font-stretch, font-variant or font-weight
-		if (node.type == 'Identifier') {
+		if (node.type === 'Identifier') {
 			if (SIZE_KEYWORDS.has(node.name)) {
 				font_size = node.name
 				return