Skip to content

Conversation

@ybiquitous
Copy link
Member

Which issue, if any, is this issue related to?

None.

Is there anything in the PR that needs further explanation?

The flat/recommended (recommended) config of eslint-plugin-n considers both CommonJS and ES modules. I believe this is more flexible for our Stylelint repositories.

Ref https://github.com/eslint-community/eslint-plugin-n/blob/v17.17.0/README.md#-configs

…ag/recommended` for CJS and ESM

The `flat/recommended` (`recommended`) config of `eslint-plugin-n` considers both CommonJS and ES modules.
I believe this is more flexible for our Stylelint repositories.

Ref <https://github.com/eslint-community/eslint-plugin-n/blob/v17.17.0/README.md#-configs>
Copy link
Member

@jeddy3 jeddy3 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you.

(Feel free to release as 24.0.0)

@ybiquitous ybiquitous added this pull request to the merge queue Apr 1, 2025
Merged via the queue into main with commit 7b6b7e6 Apr 1, 2025
5 checks passed
@ybiquitous ybiquitous deleted the support-both-esm-and-cjs-with-eslint-plugin-n branch April 1, 2025 16:07
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.

3 participants