Rm eslint rules, todos

Author: aliu39

packages/launchdarkly/.eslintrc.js

@@ -21,13 +21,7 @@ module.exports = {
     {
       files: ['test/**/*.ts'],
 
-      rules: {
-        // most of these errors come from `new Promise(process.nextTick)`
-        '@typescript-eslint/unbound-method': 'off',
-        // TODO: decide if we want to enable this again after the migration
-        // We can take the freedom to be a bit more lenient with tests
-        '@typescript-eslint/no-floating-promises': 'off',
-      },
+      rules: {},
     },
     {
       files: ['src/types/deprecated.ts'],

packages/launchdarkly/src/types.ts

@@ -1 +1 @@
-export type LaunchDarklyOptions = Record<string, never>; //TODO:
+export type LaunchDarklyOptions = Record<string, never>; // Extend as needed.

packages/utils/src/flags.ts

@@ -17,7 +17,7 @@ export const FLAG_BUFFER_SIZE = 100;
  *   oldest inserted flag is evicted.
  */
 export function insertToFlagBuffer(flags: FeatureFlag[], name: string, value: boolean): void {
-  // Check if the flag is already in the buffer
+  // Check if the flag is already in the buffer - O(n)
   const index = flags.findIndex(f => f.flag === name);
 
   if (index !== -1) {