Add configuration for @typescript-eslint/no-unused-vars.

Review Request #13271 — Created Sept. 12, 2023 and submitted

Information

beanbag-eslint-plugin
master

Reviewers

When running on TypeScript files, the basic eslint no-unused-vars plugin
is replaced by a TS version, which didn't have any configuration. This
was causing errors on things like unused function parameters, which we
don't want (we often declare function signatures in parent classes which
are implemented in child classes).

This change adds the necessary configuration for this to work.

Saw that I no longer got errors about unused function parameters when
working with TypeScript files.

Summary ID
Add configuration for @typescript-eslint/no-unused-vars.
When running on TypeScript files, the basic eslint no-unused-vars plugin is replaced by a TS version, which didn't have any configuration. This was causing errors on things like unused function parameters, which we don't want (we often declare function signatures in parent classes which are implemented in child classes). This change adds the necessary configuration for this to work. Testing Done: Saw that I no longer got errors about unused function parameters when working with TypeScript files.
b46532752d322c59dd4741e9edb58298246edb35
Description From Last Updated

These should be listed in alphabetical order.

chipx86chipx86
chipx86
  1. 
      
  2. src/index.ts (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     

    These should be listed in alphabetical order.

    1. I'll switch these but I think it's still a good idea to group the JS and typescript versions of the same thing next to each other.

  3. 
      
david
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (ceef953)
Loading...