Skip to content

Remove unused @babel/core and eslint-plugin-jsx-a11y from eslint-plugin-cli#7174

Merged
ryancbahan merged 1 commit intomainfrom
04-02-remove-unused-eslint-deps
Apr 3, 2026
Merged

Remove unused @babel/core and eslint-plugin-jsx-a11y from eslint-plugin-cli#7174
ryancbahan merged 1 commit intomainfrom
04-02-remove-unused-eslint-deps

Conversation

@ryancbahan
Copy link
Copy Markdown
Contributor

@ryancbahan ryancbahan commented Apr 2, 2026

Summary

  • Remove @babel/core — the only babel reference is a rule set to 'off' in config.js, which doesn't require the package to be installed
  • Remove eslint-plugin-jsx-a11y — already pulled in transitively via @shopify/eslint-plugin

…-plugin-cli

Neither package is directly used: @babel/core had no consumer (the only
babel reference is a rule set to 'off'), and eslint-plugin-jsx-a11y is
already pulled in transitively via @shopify/eslint-plugin.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@dmerand dmerand left a comment

Choose a reason for hiding this comment

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

Manual verification looks good.

I checked for remaining direct references in packages/eslint-plugin-cli and found none, confirmed packages/eslint-plugin-cli/config.js still loads, and ran eslint on an example file. Removal appears safe ✅

@ryancbahan ryancbahan added this pull request to the merge queue Apr 3, 2026
Merged via the queue into main with commit bc6e197 Apr 3, 2026
25 checks passed
@ryancbahan ryancbahan deleted the 04-02-remove-unused-eslint-deps branch April 3, 2026 15: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.

2 participants