chore: eslint automatically removes unused directives

Author: mikavilpas

.eslintrc.js

@@ -10,6 +10,7 @@ module.exports = {
     ecmaVersion: 'latest',
     sourceType: 'module',
   },
+  reportUnusedDisableDirectives: true,
   ignorePatterns: [
     '.eslintrc.js',
     'packages/*/lib/',

.prettierignore

@@ -0,0 +1,2 @@
+# some tests seem to create these files in CI
+nvim-win64

packages/decorators/src/plugin/plugin.ts

@@ -96,15 +96,13 @@ function wrapper(cls: PluginWrapperConstructor, options?: PluginOptions): any {
 }
 
 // Can decorate a class with options object
-// eslint-disable-next-line import/export
+
 export function Plugin(
   outter: any
 ): (cls: PluginWrapperConstructor, options?: PluginOptions) => any;
 
-// eslint-disable-next-line import/export
 export function Plugin(outter: any): any;
 
-// eslint-disable-next-line import/export
 export function Plugin(outter: any): any {
   /**
    * Decorator should support

packages/neovim/src/api/Neovim.ts

@@ -9,23 +9,23 @@ import { ApiInfo } from '../types/ApiInfo';
 export interface UiAttachOptions {
   rgb?: boolean;
   override?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_cmdline?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_hlstate?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_linegrid?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_messages?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_multigrid?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_popupmenu?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_tabline?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_wildmenu?: boolean;
-  // eslint-disable-next-line camelcase
+
   ext_termcolors?: boolean;
 }
 
@@ -56,7 +56,7 @@ export interface Command {
   nargs: string;
   range: string;
   name: string;
-  // eslint-disable-next-line camelcase
+
   script_id: number;
   bar: boolean;
   register: boolean;
@@ -65,7 +65,7 @@ export interface Command {
   complete?: null;
   addr?: any;
   count?: any;
-  // eslint-disable-next-line camelcase
+
   complete_arg?: any;
 }
 

packages/neovim/src/api/utils/createChainableApi.ts

@@ -81,7 +81,6 @@ export function createChainableApi(
     },
 
     set: (target: any, prop: string, value: any, receiver: Promise<any>) => {
-      // eslint-disable-next-line no-param-reassign
       if (receiver && (receiver instanceof Promise || 'then' in receiver)) {
         receiver.then(obj => {
           if (prop in obj) {

packages/neovim/src/host/NvimPlugin.ts

@@ -9,7 +9,7 @@ export interface NvimPluginOptions {
 
 export interface AutocmdOptions {
   pattern?: string;
-  // eslint-disable-next-line no-eval
+
   eval?: string;
   sync?: boolean;
 }
@@ -24,7 +24,7 @@ export interface CommandOptions {
 export interface NvimFunctionOptions {
   sync?: boolean;
   range?: string;
-  // eslint-disable-next-line no-eval
+
   eval?: string;
 }
 

packages/neovim/src/plugin/autocmd.ts

@@ -2,7 +2,7 @@ import { NVIM_SYNC, NVIM_SPEC, NVIM_METHOD_NAME } from './properties';
 
 export interface AutocmdOptions {
   pattern: string;
-  // eslint-disable-next-line no-eval
+
   eval?: string;
   sync?: boolean;
 }

packages/neovim/src/plugin/function.ts

@@ -3,7 +3,7 @@ import { NVIM_SYNC, NVIM_SPEC, NVIM_METHOD_NAME } from './properties';
 export interface NvimFunctionOptions {
   sync?: boolean;
   range?: [number, number];
-  // eslint-disable-next-line no-eval
+
   eval?: string;
 }
 

packages/neovim/src/types/Spec.ts

@@ -5,7 +5,7 @@ export interface Spec {
   opts: {
     range?: string;
     nargs?: string;
-    // eslint-disable-next-line no-eval
+
     eval?: string;
     pattern?: string;
     complete?: string;