refactor: use let instead of var where applicable

src/compile-string.ts

@@ -21,7 +21,7 @@ import type { AstObject } from './parse'
 export default function compileToString(str: string, config: EtaConfig): string {
   const buffer: Array<AstObject> = Parse(str, config)
 
-  var res =
+  let res =
     "var tR='',__l,__lP" +
     (config.include ? ',include=E.include.bind(E)' : '') +
     (config.includeFile ? ',includeFile=E.includeFile.bind(E)' : '') +
@@ -41,7 +41,7 @@ export default function compileToString(str: string, config: EtaConfig): string
     (config.useWith ? '}' : '')
 
   if (config.plugins) {
-    for (var i = 0; i < config.plugins.length; i++) {
+    for (let i = 0; i < config.plugins.length; i++) {
       const plugin = config.plugins[i]
       if (plugin.processFnString) {
         res = plugin.processFnString(res, config)
@@ -66,9 +66,9 @@ export default function compileToString(str: string, config: EtaConfig): string
  */
 
 function compileScope(buff: Array<AstObject>, config: EtaConfig) {
-  var i = 0
+  let i = 0
   const buffLength = buff.length
-  var returnStr = ''
+  let returnStr = ''
 
   for (i; i < buffLength; i++) {
     const currentBlock = buff[i]
@@ -79,7 +79,7 @@ function compileScope(buff: Array<AstObject>, config: EtaConfig) {
       returnStr += "tR+='" + str + "'\n"
     } else {
       const type = currentBlock.t // ~, s, !, ?, r
-      var content = currentBlock.val || ''
+      let content = currentBlock.val || ''
 
       if (type === 'r') {
         // raw

src/compile.ts

@@ -29,7 +29,7 @@ export type TemplateFunction = (data: object, config: EtaConfig, cb?: CallbackFn
 
 export default function compile(str: string, config?: PartialConfig): TemplateFunction {
   const options: EtaConfig = getConfig(config || {})
-  var ctor // constructor
+  let ctor // constructor
 
   /* ASYNC HANDLING */
   // The below code is modified from mde/ejs. All credit should go to them.

src/file-handlers.ts

@@ -193,8 +193,8 @@ function renderFile(
   And we want to also make (filename, data, options, cb) available
   */
 
-  var renderConfig: EtaConfigWithFilename
-  var callback: CallbackFn | undefined
+  let renderConfig: EtaConfigWithFilename
+  let callback: CallbackFn | undefined
   data = data || {} // If data is undefined, we don't want accessing data.settings to error
 
   // First, assign our callback function to `callback`

src/file-utils.ts

@@ -24,7 +24,7 @@ import type { EtaConfig } from './config'
  */
 
 function getWholeFilePath(name: string, parentfile: string, isDirectory?: boolean): string {
-  var includePath = path.resolve(
+  let includePath = path.resolve(
     isDirectory ? parentfile : path.dirname(parentfile), // returns directory the parent file is in
     name // file
   )
@@ -54,9 +54,9 @@ function getWholeFilePath(name: string, parentfile: string, isDirectory?: boolea
  */
 
 function getPath(path: string, options: EtaConfig): string {
-  var includePath: string | false = false
+  let includePath: string | false = false
   const views = options.views
-  var searchedPaths: Array<string> = []
+  let searchedPaths: Array<string> = []
 
   // If these four values are the same,
   // getPath() will return the same result every time.
@@ -90,7 +90,7 @@ function getPath(path: string, options: EtaConfig): string {
    */
 
   function searchViews(views: Array<string> | string | undefined, path: string): string | false {
-    var filePath
+    let filePath
 
     // If views is an array, then loop through each directory
     // And attempt to find the template

src/parse.ts

@@ -30,13 +30,13 @@ function escapeRegExp(string: string) {
 }
 
 export default function parse(str: string, config: EtaConfig): Array<AstObject> {
-  var buffer: Array<AstObject> = []
-  var trimLeftOfNextStr: string | false = false
-  var lastIndex = 0
+  let buffer: Array<AstObject> = []
+  let trimLeftOfNextStr: string | false = false
+  let lastIndex = 0
   const parseOptions = config.parse
 
   if (config.plugins) {
-    for (var i = 0; i < config.plugins.length; i++) {
+    for (let i = 0; i < config.plugins.length; i++) {
       const plugin = config.plugins[i]
       if (plugin.processTemplate) {
         str = plugin.processTemplate(str, config)
@@ -108,7 +108,7 @@ export default function parse(str: string, config: EtaConfig): Array<AstObject>
   )
   // TODO: benchmark having the \s* on either side vs using str.trim()
 
-  var m
+  let m
 
   while ((m = parseOpenReg.exec(str))) {
     lastIndex = m[0].length + m.index
@@ -120,18 +120,18 @@ export default function parse(str: string, config: EtaConfig): Array<AstObject>
     pushString(precedingString, wsLeft)
 
     parseCloseReg.lastIndex = lastIndex
-    var closeTag
-    var currentObj: AstObject | false = false
+    let closeTag
+    let currentObj: AstObject | false = false
 
     while ((closeTag = parseCloseReg.exec(str))) {
       if (closeTag[1]) {
-        var content = str.slice(lastIndex, closeTag.index)
+        let content = str.slice(lastIndex, closeTag.index)
 
         parseOpenReg.lastIndex = lastIndex = parseCloseReg.lastIndex
 
         trimLeftOfNextStr = closeTag[2]
 
-        var currentType: TagType = ''
+        let currentType: TagType = ''
         if (prefix === parseOptions.exec) {
           currentType = 'e'
         } else if (prefix === parseOptions.raw) {
@@ -190,7 +190,7 @@ export default function parse(str: string, config: EtaConfig): Array<AstObject>
   pushString(str.slice(lastIndex, str.length), false)
 
   if (config.plugins) {
-    for (var i = 0; i < config.plugins.length; i++) {
+    for (let i = 0; i < config.plugins.length; i++) {
       const plugin = config.plugins[i]
       if (plugin.processAST) {
         buffer = plugin.processAST(buffer, config)

src/render.ts

@@ -12,7 +12,7 @@ import type { CallbackFn } from './file-handlers'
 /* END TYPES */
 
 function handleCache(template: string | TemplateFunction, options: EtaConfig): TemplateFunction {
-  var templateFunc
+  let templateFunc
 
   if (options.cache && options.name && options.templates.get(options.name)) {
     return options.templates.get(options.name)
@@ -61,7 +61,7 @@ export default function render(
   const options = getConfig(config || {})
 
   if (options.async) {
-    var result
+    let result
     if (cb) {
       // If user passes callback
       try {

src/utils.ts

@@ -41,8 +41,8 @@ function trimWS(
   wsLeft: string | false,
   wsRight?: string | false
 ): string {
-  var leftTrim
-  var rightTrim
+  let leftTrim
+  let rightTrim
 
   if (Array.isArray(config.autoTrim)) {
     // kinda confusing