import-js / import-js/eslint-plugin-import
Add ignore pattern before using unusedExports by default
- 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
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