Skip to content

add eslint-plugin-unused-imports - #18

Merged
lukasvice merged 2 commits into
mainfrom
eslint-plugin-unused-imports
Aug 14, 2025
Merged

add eslint-plugin-unused-imports#18
lukasvice merged 2 commits into
mainfrom
eslint-plugin-unused-imports

Conversation

@lukasvice

Copy link
Copy Markdown
Member

This plugin detects and removes unused imports so that we are not solely reliant on IDE settings.

See https://github.com/sweepline/eslint-plugin-unused-imports

@lukasvice
lukasvice force-pushed the eslint-plugin-unused-imports branch from 790a7f1 to 683371b Compare August 13, 2025 14:17
@lukasvice
lukasvice requested a review from Copilot August 13, 2025 14:17

This comment was marked as outdated.

@lukasvice
lukasvice requested a review from Copilot August 14, 2025 06:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds the eslint-plugin-unused-imports plugin to automatically detect and remove unused imports, reducing reliance on IDE settings for this functionality. The changes configure ESLint to use this plugin for better import management.

  • Adds eslint-plugin-unused-imports dependency to package.json
  • Configures ESLint to use the new plugin and its rules
  • Replaces the existing TypeScript unused vars rule with the plugin's equivalent functionality

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
ts.js Updates ESLint configuration to include the unused-imports plugin and replaces existing unused vars rule
package.json Adds eslint-plugin-unused-imports dependency to both dependencies and peerDependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@SirCotare SirCotare left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! 👍

@lukasvice
lukasvice merged commit ed6d62e into main Aug 14, 2025
@lukasvice
lukasvice deleted the eslint-plugin-unused-imports branch August 14, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants