import-js / import-js/eslint-plugin-import

Add ignore pattern before using unusedExports by default

Open
#2,970 8 comments 0 reactions 0 assignees View on GitHub
enhancement rule proposal
Dominant language
JavaScript
Stars
5.9k
Forks
1.5k
PR merge metrics
No merged PRs in 30d

Description

Before doing https://github.com/import-js/eslint-plugin-import/issues/1324 We should have a `varsIgnorePattern` or similar. There is cases where you don't want to remove all unsued exports.
For example in some project when we have a custom library / DS we export the props from the component files. Though we do not want to ignore the files itself, only interface / type that end with `Props`

Contributor guide

Open the contributing guide

Research direction

Start by reading the implementation and tests for the unusedExports rule, especially how exported variables are selected. Define the ignore-pattern behavior for names ending in Props without ignoring entire files, then add coverage showing those exports are preserved while other unused exports remain removable.

Written by the indexing model from the issue text.

Assessment

Tech stack
eslint, javascript
Domain
tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 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.