Skip to content

Conversation

michaelfaith
Copy link
Contributor

This change is in support of the larger migration to TypeScript, and a followup to the removal of support for ESLint v8. The context compatibility functions in the utils module were no longer necessary. I originally did this in the TypeScript branch, but that PR's going to be large enough. So, I'm trying to peel off smaller independent changes I can land separately first.

I also took the opportunity to shift the rules to use named imports from utils.

This change is in support of the larger migration to TypeScript., and a followup to the removal of support for ESLint v8.  The context compatibility functions in the `utils` module were no longer necessary.  I originally did this in the TypeScript branch, but that PR's going to be large enough.  So, I'm trying to peel off smaller independent changes I can land separately first.
@michaelfaith michaelfaith marked this pull request as ready for review July 22, 2025 23:10
@bmish bmish added the BREAKING CHANGE This change will require a major version bump label Jul 23, 2025
Copy link
Member

@bmish bmish left a comment

Choose a reason for hiding this comment

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

Thanks!

@bmish bmish merged commit f834dc2 into eslint-community:main Jul 23, 2025
5 checks passed
@michaelfaith michaelfaith deleted the refactor/remove-context-compat branch July 23, 2025 01:41
bmish added a commit to bmish/eslint-plugin-eslint-plugin that referenced this pull request Aug 4, 2025
* main:
  feat: migrate package to TypeScript and publish types (eslint-community#534)
  build: convert eslint-remote-tester config to typescript (eslint-community#533)
  refactor: remove context compat functions (eslint-community#532)
  build: migrate eslint.config.js to typescript  (eslint-community#531)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BREAKING CHANGE This change will require a major version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants