Update linting rule (#90)

* update linting rule

* fix format issue

* update

Author: zhiyuanliang-ms

.eslintrc

@@ -40,6 +40,22 @@
                 "avoidEscape": true
             }
         ],
-        "@typescript-eslint/no-explicit-any": "off"
+        "@typescript-eslint/no-explicit-any": "off",
+        "eol-last": [
+            "error",
+            "always"
+        ],
+        "no-trailing-spaces": "error",
+        "space-before-blocks": [
+            "error",
+            "always"
+        ],
+        "no-multi-spaces": "error",
+        "no-multiple-empty-lines": [
+            "error",
+            {
+                "max": 1 
+            }
+        ]
     }
 }

src/AzureAppConfigurationImpl.ts

@@ -575,4 +575,4 @@ function getValidFeatureFlagSelectors(selectors?: SettingSelector[]): SettingSel
     } else {
         return getValidSelectors(selectors);
     }
-}
+}

src/AzureAppConfigurationOptions.ts

@@ -47,4 +47,4 @@ export interface AzureAppConfigurationOptions {
      * Specifies options used to configure feature flags.
      */
     featureFlagOptions?: FeatureFlagOptions;
-}
+}

src/IKeyValueAdapter.ts

@@ -13,4 +13,4 @@ export interface IKeyValueAdapter {
      * This method process the original configuration setting, and returns processed key and value in an array.
      */
     processKeyValue(setting: ConfigurationSetting): Promise<[string, unknown]>;
-}
+}

src/JsonKeyValueAdapter.ts

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

src/RefreshOptions.ts

@@ -20,7 +20,7 @@ export interface RefreshOptions {
     refreshIntervalInMs?: number;
 
     /**
-     * One or more configuration settings to be watched for changes on the server. 
+     * One or more configuration settings to be watched for changes on the server.
      * Any modifications to watched settings will refresh all settings loaded by the configuration provider when refresh() is called.
      */
     watchedSettings?: WatchedSetting[];

src/WatchedSetting.ts

@@ -15,4 +15,4 @@ export interface WatchedSetting {
      * Leaving this undefined means this setting does not have a label.
      */
     label?: string;
-}
+}

src/common/disposable.ts

@@ -15,4 +15,4 @@ export class Disposable {
         }
         this.#disposed = true;
     }
-}
+}

src/featureManagement/FeatureFlagOptions.ts

@@ -27,4 +27,4 @@ export interface FeatureFlagOptions {
      * Specifies how feature flag refresh is configured. All selected feature flags will be watched for changes.
      */
     refresh?: FeatureFlagRefreshOptions;
-}
+}

src/featureManagement/constants.ts

@@ -2,4 +2,4 @@
 // Licensed under the MIT license.
 
 export const FEATURE_MANAGEMENT_KEY_NAME = "feature_management";
-export const FEATURE_FLAGS_KEY_NAME = "feature_flags";
+export const FEATURE_FLAGS_KEY_NAME = "feature_flags";

src/keyvault/AzureKeyVaultKeyValueAdapter.ts

@@ -72,4 +72,4 @@ export class AzureKeyVaultKeyValueAdapter implements IKeyValueAdapter {
 
 function getHost(url: string) {
     return new URL(url).host;
-}
+}

src/keyvault/KeyVaultOptions.ts

@@ -24,4 +24,4 @@ export interface KeyVaultOptions {
      * @returns The secret value.
      */
     secretResolver?: (keyVaultReference: URL) => string | Promise<string>;
-}
+}

src/version.ts

@@ -1,4 +1,4 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 
-export const VERSION = "1.1.0";
+export const VERSION = "1.1.0";

test/exportedApi.ts

@@ -1,4 +1,4 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 
-export { load } from "../src";
+export { load } from "../src";

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/refresh.test.ts

@@ -423,4 +423,4 @@ describe("dynamic refresh feature flags", function () {
         await settings.refresh();
         expect(refreshSuccessfulCount).eq(1); // change in feature flags, because page etags are different.
     });
-});
+});

test/requestTracing.test.ts

@@ -345,4 +345,4 @@ describe("request tracing", function () {
             expect(correlationContext.includes("Host=Web")).eq(false);
         });
     });
-});
+});

test/utils/testHelper.ts

@@ -210,4 +210,4 @@ export {
     createMockedFeatureFlag,
 
     sleepInMs
-}
+}