ignore eslint-comments/no-unused-disable linting rule

XRPLF · Nov 16, 2022 · a76b310 · a76b310
1 parent 4bc48ae
commit a76b310
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 2 deletions.
diff --git a/packages/xrpl/.eslintrc.js b/packages/xrpl/.eslintrc.js
@@ -31,6 +31,7 @@ module.exports = {
     // Disabled until https://github.com/import-js/eslint-plugin-import/pull/2305 is resolved to
     // accomodate this change https://github.com/XRPLF/xrpl.js/pull/2133
     'import/no-unused-modules': 'off',
+    'eslint-comments/no-unused-disable': 'off',
     // Certain rippled APIs require snake_case naming
     '@typescript-eslint/naming-convention': [
       'error',

diff --git a/packages/xrpl/src/client/WSWrapper.ts b/packages/xrpl/src/client/WSWrapper.ts
@@ -1,4 +1,3 @@
-/* eslint-disable import/no-unused-modules -- Used by webpack */
 /* eslint-disable max-classes-per-file -- Needs to be a wrapper for ws */
 import { EventEmitter } from 'events'
 

diff --git a/packages/xrpl/src/models/methods/subscribe.ts b/packages/xrpl/src/models/methods/subscribe.ts
@@ -136,7 +136,6 @@ export interface LedgerStream extends BaseStream {
 /**
  * This response mirrors the LedgerStream, except it does NOT include the 'type' nor 'txn_count' fields.
  */
-// eslint-disable-next-line import/no-unused-modules -- Detailed enough to be worth exporting for end users.
 export interface LedgerStreamResponse {
   /**
    * The reference transaction cost as of this ledger version, in drops of XRP.