search

aertslab / SCope

Fast visualization tool for large-scale and high dimensional single-cell data
GNU General Public License v3.0
68 stars 15 forks source link

Add the Eslint rule: no-unused-vars #460

Closed MaybeJustJames closed 3 years ago

MaybeJustJames commented 3 years ago

The change adds the ESLint rule no-unused-vars (https://eslint.org/docs/rules/no-unused-vars). This was added because a common reviewer comment was to remove unused imports and variables, this rule automates that review work. This change also makes all existing client code pass this check.

MaybeJustJames commented 3 years ago

@dweemx I cancelled the Server checks. They ran because I accidentally selected master as the target branch rather than develop. This PR doesn't touch server code.