Update linting rule to enforce to use semicolon (#91)

* enforce to use semicolon

* fix lint

Author: zhiyuanliang-ms

.eslintrc

@@ -56,6 +56,7 @@
             {
                 "max": 1 
             }
-        ]
+        ],
+        "semi": ["error", "always"]
     }
 }

src/AzureAppConfigurationImpl.ts

@@ -481,7 +481,7 @@ export class AzureAppConfigurationImpl implements AzureAppConfiguration {
             if (index >= 0) {
                 this.#onRefreshListeners.splice(index, 1);
             }
-        }
+        };
         return new Disposable(remove);
     }
 

src/JsonKeyValueAdapter.ts

@@ -54,4 +54,4 @@ function isJsonContentType(contentTypeValue: string): boolean {
     }
 
     return typeParts[1].split("+").includes("json");
-}
+}

src/load.ts

@@ -94,7 +94,7 @@ function getClientOptions(options?: AzureAppConfigurationOptions): AppConfigurat
     const defaultRetryOptions = {
         maxRetries: MaxRetries,
         maxRetryDelayInMs: MaxRetryDelayInMs,
-    }
+    };
     const retryOptions = Object.assign({}, defaultRetryOptions, options?.clientOptions?.retryOptions);
 
     return Object.assign({}, options?.clientOptions, {

src/requestTracing/utils.ts

@@ -40,7 +40,7 @@ export function listConfigurationSettingsWithTrace(
             customHeaders: {
                 [CORRELATION_CONTEXT_HEADER_NAME]: createCorrelationContextHeader(appConfigOptions, initialLoadCompleted)
             }
-        }
+        };
     }
 
     return client.listConfigurationSettings(actualListOptions);
@@ -64,7 +64,7 @@ export function getConfigurationSettingWithTrace(
             customHeaders: {
                 [CORRELATION_CONTEXT_HEADER_NAME]: createCorrelationContextHeader(appConfigOptions, initialLoadCompleted)
             }
-        }
+        };
     }
 
     return client.getConfigurationSetting(configurationSettingId, actualGetOptions);

test/clientOptions.test.ts

@@ -37,7 +37,7 @@ describe("custom client options", function () {
 
     afterEach(() => {
         nock.restore();
-    })
+    });
 
     it("should retry 2 times by default", async () => {
         const countPolicy = new HttpRequestCountPolicy();
@@ -49,7 +49,7 @@ describe("custom client options", function () {
                         position: "perRetry"
                     }]
                 }
-            })
+            });
         };
         let error;
         try {
@@ -74,7 +74,7 @@ describe("custom client options", function () {
                         maxRetries
                     }
                 }
-            })
+            });
         };
 
         let error;
@@ -109,7 +109,7 @@ describe("custom client options", function () {
                         position: "perRetry"
                     }]
                 }
-            })
+            });
         };
         let error;
         try {
@@ -120,4 +120,4 @@ describe("custom client options", function () {
         expect(error).not.undefined;
         expect(countPolicy.count).eq(3);
     });
-})
+});

test/featureFlag.test.ts

@@ -65,7 +65,8 @@ describe("feature flags", function () {
 
     after(() => {
         restoreMocks();
-    })
+    });
+
     it("should load feature flags if enabled", async () => {
         const connectionString = createMockedConnectionString();
         const settings = await load(connectionString, {

test/json.test.ts

@@ -17,7 +17,7 @@ describe("json", function () {
 
     afterEach(() => {
         restoreMocks();
-    })
+    });
 
     it("should load and parse if content type is application/json", async () => {
         mockAppConfigurationClientListConfigurationSettings([jsonKeyValue]);
@@ -86,4 +86,4 @@ describe("json", function () {
         expect(settings.get("json.settings.emptyString")).eq("", "is empty string");
         expect(settings.get("json.settings.illegalString")).eq("[unclosed", "is illegal string");
     });
-})
+});

test/keyvault.test.ts

@@ -111,4 +111,4 @@ describe("key vault reference", function () {
         expect(settings.get("TestKey")).eq("SecretValue");
         expect(settings.get("TestKey2")).eq("SecretValue2");
     });
-})
+});

test/load.test.ts

@@ -85,7 +85,8 @@ describe("load", function () {
 
     after(() => {
         restoreMocks();
-    })
+    });
+
     it("should load data from config store with connection string", async () => {
         const connectionString = createMockedConnectionString();
         const settings = await load(connectionString);