Automattic / Automattic/woocommerce-services

Upgrade eslint to latest version and fix all linting problems

Open
#2,097 0 comments 1 reaction 0 assignees View on GitHub
[Pri] High [Type] Good First Change [Type] Technical Debt woocommerce-services
Dominant language
JavaScript
Stars
112
Forks
31
Avg merge
3d 1h
Merged PRs (30d)
14

Description

Current eslint version [has a bug](https://github.com/eslint/eslint/issues/12117) that will give false positives about variables defined inside `for` loops not being used. Upgrading to the latest eslint version ( and, apparently, removing the eslint cache too) should get rid of this problem.

Also, all current eslint errors should be addressed too.

Discussed here: https://github.com/Automattic/woocommerce-services/pull/2094

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the project's ESLint version and cache configuration, then inspect the current lint output. The discussion in pull request 2094 and the linked ESLint bug provide context for the upgrade. Done means upgrading ESLint, clearing the cache if needed, and addressing all current linting errors.

Written by the indexing model from the issue text.

Assessment

Tech stack
eslint, javascript
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.