refactor: update scripts to remove eslint warnings

Author: mrholek

build/postinstall.js

@@ -15,7 +15,7 @@ const main = () => {
 
   try {
     const Configstore = require('configstore')
-    const pkg = require(__dirname + '/../package.json')
+    const pkg = require(`${__dirname}/../package.json`)
     const now = Date.now()
 
     const week = 1000 * 60 * 60 * 24 * 7

build/vendors.js

@@ -11,19 +11,20 @@ const injector = require('@coreui/vendors-injector')
 const dist = 'dist/'
 
 const walkSync = (dir, filelist = []) => {
-  fs.readdirSync(dir).forEach(file => {
+  for (const file of fs.readdirSync(dir)) {
     filelist = fs.statSync(path.join(dir, file)).isDirectory() ? walkSync(path.join(dir, file), filelist) : filelist.concat(path.join(dir, file))
-  })
+  }
+
   return filelist
 }
 
 const main = () => {
   const filenames = walkSync(dist)
-  filenames.forEach(filename => {
+  for (const filename of filenames) {
     if (extension(filename) === '.html') {
       injector.toFile(filename)
     }
-  })
+  }
 }
 
 main()

src/js/colors.js

@@ -7,7 +7,7 @@
  * --------------------------------------------------------------------------
  */
 
-document.querySelectorAll('.theme-color').forEach(element => {
+for (const element of document.querySelectorAll('.theme-color')) {
   const color = getComputedStyle(element, null).getPropertyValue('background-color')
   const table = document.createElement('table')
   table.classList.add('w-100')
@@ -24,4 +24,4 @@ document.querySelectorAll('.theme-color').forEach(element => {
       </table>
     `
   element.parentNode.append(table)
-})
+}

src/js/popovers.js

@@ -7,8 +7,8 @@
  * --------------------------------------------------------------------------
  */
 
-document.querySelectorAll('[data-coreui-toggle="popover"]').forEach(element => {
+for (const element of document.querySelectorAll('[data-coreui-toggle="popover"]')) {
   // eslint-disable-next-line no-new
   new coreui.Popover(element)
-})
+}
 

src/js/tooltips.js

@@ -7,7 +7,7 @@
  * --------------------------------------------------------------------------
  */
 
-document.querySelectorAll('[data-coreui-toggle="tooltip"]').forEach(element => {
+for (const element of document.querySelectorAll('[data-coreui-toggle="tooltip"]')) {
   // eslint-disable-next-line no-new
   new coreui.Tooltip(element)
-})
+}