Skip to content

doc: remove unnecessary <!-- eslint- magic comments#63200

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
aduh95:eslint-everywhere
May 10, 2026
Merged

doc: remove unnecessary <!-- eslint- magic comments#63200
nodejs-github-bot merged 1 commit intonodejs:mainfrom
aduh95:eslint-everywhere

Conversation

@aduh95
Copy link
Copy Markdown
Contributor

@aduh95 aduh95 commented May 8, 2026

Some of them are no longer needed, some of them have probably never been needed, and some of them were ignoring actual problems in the associated snippet.

Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto
  • @nodejs/http
  • @nodejs/http2
  • @nodejs/loaders
  • @nodejs/net
  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label May 8, 2026
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels May 8, 2026
@idabeck-cpu

This comment was marked as spam.

@idabeck-cpu

This comment was marked as spam.

@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 10, 2026
@nodejs-github-bot nodejs-github-bot merged commit 49aef0d into nodejs:main May 10, 2026
28 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 49aef0d

@aduh95 aduh95 deleted the eslint-everywhere branch May 10, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants