Skip to content

chore(Examples): Drop local eslint plugin#4260

Open
t0maboro wants to merge 1 commit into
mainfrom
@t0maboro/drop-eslint-plugin
Open

chore(Examples): Drop local eslint plugin#4260
t0maboro wants to merge 1 commit into
mainfrom
@t0maboro/drop-eslint-plugin

Conversation

@t0maboro

@t0maboro t0maboro commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description

No longer needed after #4201 . Reverting to the state before #3959 .

Changes

  • removing all files and references to no longer used plugin

Before & after - visual documentation

N/A

Test plan

yarn lint is passing

Checklist

  • Included code example that can be used to test this change.
  • For visual changes, included screenshots / GIFs / recordings documenting the change.
  • For API changes, updated relevant public types.
  • Ensured that CI passes

@t0maboro t0maboro changed the title chore(lib): Drop local eslint plugin chore(Examples): Drop local eslint plugin Jul 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes the now-unneeded local ESLint plugin from the example apps workspace, reverting the examples’ lint setup to the state before the custom require-top-level-exports rule was introduced.

Changes:

  • Drops eslint-plugin-local-rules from apps/package.json and prunes it from apps/yarn.lock.
  • Removes the tests-specific ESLint override config that enabled the local rule.
  • Deletes the local ESLint plugin implementation from apps/eslint-plugin-local-rules/.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
apps/yarn.lock Removes the workspace link/locator entry for the deleted local ESLint plugin.
apps/src/tests/.eslintrc.js Deletes the tests override that enabled the local-rules/require-top-level-exports rule.
apps/package.json Removes the eslint-plugin-local-rules devDependency.
apps/eslint-plugin-local-rules/require-top-level-exports.js Deletes the custom rule implementation.
apps/eslint-plugin-local-rules/package.json Deletes the plugin’s package manifest.
apps/eslint-plugin-local-rules/index.js Deletes the plugin entry point exporting the rule.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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