chore: clean up unused ignore directives

Author: mrstork

packages/blobs/src/store.ts

@@ -107,7 +107,7 @@ export class Store {
   async get(key: string, opts: GetOptions): Promise<string>
   async get(key: string, { type }: GetOptions & { type: 'arrayBuffer' }): Promise<ArrayBuffer>
   async get(key: string, { type }: GetOptions & { type: 'blob' }): Promise<Blob>
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+   
   async get(key: string, { type }: GetOptions & { type: 'json' }): Promise<any>
   async get(key: string, { type }: GetOptions & { type: 'stream' }): Promise<ReadableStream>
   async get(key: string, { type }: GetOptions & { type: 'text' }): Promise<string>
@@ -185,14 +185,14 @@ export class Store {
     options: { type: 'blob' } & GetWithMetadataOptions,
   ): Promise<({ data: Blob } & GetWithMetadataResult) | null>
 
-  /* eslint-disable @typescript-eslint/no-explicit-any */
+   
 
   async getWithMetadata(
     key: string,
     options: { type: 'json' } & GetWithMetadataOptions,
   ): Promise<({ data: any } & GetWithMetadataResult) | null>
 
-  /* eslint-enable @typescript-eslint/no-explicit-any */
+   
 
   async getWithMetadata(
     key: string,
@@ -276,7 +276,6 @@ export class Store {
 
     // We can't use `async/await` here because that would make the signature
     // incompatible with one of the overloads.
-    // eslint-disable-next-line promise/prefer-await-to-then
     return collectIterator(iterator).then((items) =>
       items.reduce(
         (acc, item) => ({

packages/blobs/src/store_factory.ts

@@ -27,7 +27,6 @@ export const getDeployStore = (input: GetDeployStoreOptions | string = {}): Stor
     // If a region hasn't been supplied and we're dealing with an edge request,
     // use the region from the context if one is defined, otherwise throw.
     if (clientOptions.edgeURL || clientOptions.uncachedEdgeURL) {
-      // eslint-disable-next-line max-depth
       if (!context.primaryRegion) {
         throw new Error(
           'When accessing a deploy store, the Netlify Blobs client needs to be configured with a region, and one was not found in the environment. To manually set the region, set the `region` property in the `getDeployStore` options. If you are using the Netlify CLI, you may have an outdated version; run `npm install -g netlify-cli@latest` to update and try again.',

packages/blobs/src/store_list.ts

@@ -21,7 +21,6 @@ export function listStores(
 
   // We can't use `async/await` here because that would make the signature
   // incompatible with one of the overloads.
-  // eslint-disable-next-line promise/prefer-await-to-then
   return collectIterator(iterator).then((results) => ({ stores: results.flatMap((page) => page.stores) }))
 }
 

packages/blobs/test/util.ts

@@ -7,7 +7,7 @@ export const base64Encode = (input: string | object) => {
 }
 
 export const streamToString = async function streamToString(stream: NodeJS.ReadableStream): Promise<string> {
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+   
   const chunks: Array<any> = []
 
   for await (const chunk of stream) {

packages/cache/src/bootstrap/cache.ts

@@ -101,7 +101,7 @@ export class NetlifyCache implements Cache {
     await Promise.allSettled(requests.map((request) => this.add(request)))
   }
 
-  // eslint-disable-next-line class-methods-use-this, require-await, @typescript-eslint/no-unused-vars
+   
   async delete(request: RequestInfo | URL) {
     const context = this.#getContext({ operation: Operation.Delete })
 
@@ -117,7 +117,6 @@ export class NetlifyCache implements Cache {
     return true
   }
 
-  // eslint-disable-next-line class-methods-use-this, require-await, @typescript-eslint/no-unused-vars
   async keys(_request?: Request | URL) {
     // Not implemented.
     return []
@@ -151,7 +150,6 @@ export class NetlifyCache implements Cache {
     }
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   async matchAll(request?: RequestInfo | URL, _options?: CacheQueryOptions): Promise<readonly Response[]> {
     if (!request) {
       return []

packages/dev-utils/src/lib/global-config.ts

@@ -9,14 +9,14 @@ import { sync as writeFileAtomicSync } from 'write-file-atomic'
 import { getLegacyPathInHome, getPathInHome } from './paths.js'
 
 type ConfigStoreOptions<
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+   
   T extends Record<string, any>,
 > = {
   defaults?: T | undefined
 }
 
 export class GlobalConfigStore<
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+   
   T extends Record<string, any> = Record<string, any>,
 > {
   #storagePath: string
@@ -41,7 +41,7 @@ export class GlobalConfigStore<
   }
 
   public get(key: string): T[typeof key] {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
+     
     return dot.getProperty(this.getConfig(), key)
   }
 

packages/dev-utils/src/lib/local-state.ts

@@ -87,7 +87,7 @@ export class LocalState {
       // TODO figure out cleaner way of grabbing ENV vars
       return process.env.NETLIFY_SITE_ID
     }
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression
+     
     return getProperty(this.all, key)
   }
 

packages/dev-utils/src/lib/memoize.ts

@@ -22,11 +22,10 @@ interface MemoizeOptions<T> {
 // This allows us to discard any duplicate filesystem events, while ensuring
 // that actual updates happening during the zip operation will be executed
 // after it finishes (only the last update will run).
-/* eslint-disable no-param-reassign */
+ 
 export const memoize = <T>({ cache, cacheKey, command }: MemoizeOptions<T>) => {
   if (cache[cacheKey] === undefined) {
     cache[cacheKey] = {
-      // eslint-disable-next-line promise/prefer-await-to-then
       task: command().finally(() => {
         const entry = cache[cacheKey]
 
@@ -44,4 +43,4 @@ export const memoize = <T>({ cache, cacheKey, command }: MemoizeOptions<T>) => {
 
   return cache[cacheKey].task
 }
-/* eslint-enable no-param-reassign */
+ 

packages/dev-utils/src/test/fetch.ts

@@ -64,7 +64,7 @@ export class MockFetch {
   }
 
   get fetch() {
-    // eslint-disable-next-line require-await
+     
     return async (...args: Parameters<typeof globalThis.fetch>) => {
       const [reqOrURL, options] = args
       const url = (reqOrURL instanceof Request ? reqOrURL.url : reqOrURL).toString()

packages/functions/dev/function.ts

@@ -159,7 +159,6 @@ export class NetlifyFunction {
 
   hasValidName() {
     // same as https://github.com/netlify/bitballoon/blob/fbd7881e6c8e8c48e7a0145da4ee26090c794108/app/models/deploy.rb#L482
-    // eslint-disable-next-line unicorn/better-regex
     return /^[A-Za-z0-9_-]+$/.test(this.name)
   }
 
@@ -186,7 +185,7 @@ export class NetlifyFunction {
       return null
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+     
     return getNextRun(this.schedule!)
   }