fix: ESLint array-callback-return

- .filter must return a boolean value
- .map replaced with forEach when the value isn't used

Author: nschonni

lib/remove-liquid-statements.js

@@ -99,6 +99,7 @@ function removeLiquidOnly (content, allLiquidBlocks, regexes) {
       if (block.match(regexes.andGreaterThanVersionToDeprecate2)) return firstIf[0] === block.match(regexes.andGreaterThanVersionToDeprecate2)[0]
       if (block.match(regexes.notEqualsVersionToDeprecate)) return firstIf[0] === block.match(regexes.notEqualsVersionToDeprecate)[0]
       if (block.match(regexes.andNotEqualsVersionToDeprecate)) return firstIf[0] === block.match(regexes.andNotEqualsVersionToDeprecate)[0]
+      return false
     })
 
   blocksToUpdate.forEach(block => {
@@ -182,6 +183,7 @@ function removeLiquidAndContent (content, allLiquidBlocks, regexes) {
       const firstIf = block.match(firstIfRegex)
       if (block.match(regexes.lessThanNextOldestVersion)) return firstIf[0] === block.match(regexes.lessThanNextOldestVersion)[0]
       if (block.match(regexes.equalsVersionToDeprecate)) return firstIf[0] === block.match(regexes.equalsVersionToDeprecate)[0]
+      return false
     })
 
   blocksToRemove.forEach(block => {

lib/site-tree.js

@@ -18,14 +18,14 @@ const allVersions = Object.keys(require('./all-versions'))
 module.exports = async function buildSiteTree (pages, site, redirects) {
   const siteTree = {}
 
-  languageCodes.map(languageCode => {
+  languageCodes.forEach(languageCode => {
     siteTree[languageCode] = {}
 
-    allVersions.map(version => {
+    allVersions.forEach(version => {
       siteTree[languageCode][version] = {}
       const productTree = {}
 
-      products.map(item => {
+      products.forEach(item => {
         const product = { title: item.name }
 
         // return early if the product has external docs, like Atom
@@ -60,7 +60,7 @@ function buildCategoriesTree (tocItems, productHref, pages, redirects, version,
   const categoryTree = {}
 
   // for every category in a product TOC...
-  tocItems.map(item => {
+  tocItems.forEach(item => {
     const category = {}
 
     const categoryHref = path.join(productHref, item.href)
@@ -110,7 +110,7 @@ function buildMaptopicsTree (tocItems, categoryHref, pages, redirects, version,
   // for every maptopic in a category TOC...
   tocItems
     .filter(item => item.type === 'maptopic')
-    .map(item => {
+    .forEach(item => {
       const maptopic = {}
 
       const maptopicHref = path.join(categoryHref, item.href)
@@ -155,7 +155,7 @@ function buildArticlesTree (tocItems, categoryHref, pages, redirects, version, l
   if (!tocItems) return articleTree
 
   // for every article in a maptopic (or category) TOC...
-  tocItems.map(item => {
+  tocItems.forEach(item => {
     const article = {}
 
     const articleHref = path.join(categoryHref, item.href)
@@ -194,5 +194,6 @@ function getChildArticles (tocItems, maptopicPath) {
     } else {
       if (withinMaptopic) return item.href
     }
+    return false
   })
 }

script/update-readme.js

@@ -46,6 +46,7 @@ allScripts.forEach(script => {
       if (startCommentRegex.test(cmt)) addToReadme = true
       if (endCommentRegex.test(cmt)) addToReadme = false
       if (addToReadme && !cmt.includes(startComment) && !cmt.includes(endComment)) return cmt
+      return false
     })
     // remove comment markers and clean up newlines
     .map(cmt => cmt.replace(/^(\/\/|#) ?/m, ''))