Skip to content

chore: adopt deprecated: DeprecatedInfo meta rule option #446

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

error-four-o-four
Copy link

@error-four-o-four error-four-o-four commented Jun 8, 2025

Hey there,

eslint introduced new declarations for meta.deprecated: DeprecatedInfo

This PR updates the metadata of the rules n/shebang and n/no-hide-core-modules to use a detailed deprecation object instead of a boolean flag.

Unfortunately eslint-doc-generator hasn't implemented this yet #512. Therefore the replacedBy description is removed in shebang.md

Apart from that:
✅ files are formatted
✅ files are linted
✅ tests pass

@@ -1,6 +1,6 @@
# Require correct usage of hashbang (`n/shebang`)

❌ This rule is deprecated. It was replaced by [`n/hashbang`](hashbang.md).
❌ This rule is deprecated.

Choose a reason for hiding this comment

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

Let's just hold it -- wait for it to be supported in `eslint-doc-generator'.

@aladdin-add aladdin-add marked this pull request as draft June 10, 2025 16:21
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