add no-param-reassign, prefer-for-of, prefer-arrow-callback, and no-v…

…ar rules

.eslintrc.js

@@ -50,6 +50,10 @@ module.exports = {
         'array-callback-return': 'error',
         'no-template-curly-in-string': 'error',
         'prefer-template': 'error',
+        'no-param-reassign': 'error',
+        '@typescript-eslint/prefer-for-of': 'error',
+        'prefer-arrow-callback': 'error',
+        'no-var': 'error',
         eqeqeq: 'error',
         ...temporarilyDisabledRules
     }

examples/motoko_examples/minimal-counter-dapp/src/minimal_dapp_assets/src/index.js

@@ -1,6 +1,6 @@
 import { minimal_dapp } from '../../declarations/minimal_dapp';
 
-document.addEventListener('DOMContentLoaded', async function () {
+document.addEventListener('DOMContentLoaded', async () => {
     const counter = await minimal_dapp.getCount();
     document.getElementById('counter').innerText = `Counter: ${counter}`;
 });

src/lib/globals.ts

@@ -114,9 +114,7 @@ if (globalThis._azleInsideCanister) {
                 instantiatedSource.instance.exports
             );
 
-            for (let i = 0; i < exportEntries.length; i++) {
-                const [key, value] = exportEntries[i];
-
+            for (const [key, value] of exportEntries) {
                 if (typeof value === 'function') {
                     instantiatedSource.instance.exports[key] = value.bind({
                         instanceUuid: uuid,

test/index.ts

@@ -34,9 +34,7 @@ export async function runTests(
     tests: Test[],
     exitProcess: boolean = true
 ): Promise<boolean> {
-    for (let i = 0; i < tests.length; i++) {
-        const test = tests[i];
-
+    for (const test of tests) {
         try {
             if (test.skip === true) {
                 console.log(`Skipping: ${test.name}`);