Skip to content

doc: fix small logic error in DETECT_MODULE_SYNTAX#62025

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
Renegade334:doc-esm-detect-algo
Mar 1, 2026
Merged

doc: fix small logic error in DETECT_MODULE_SYNTAX#62025
nodejs-github-bot merged 1 commit intonodejs:mainfrom
Renegade334:doc-esm-detect-algo

Conversation

@Renegade334
Copy link
Member

Super pedantic, but the case where a source parses as ESM but doesn't contain any ESM-only syntax currently falls off the end of the algorithm without a resolution.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/loaders

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. esm Issues and PRs related to the ECMAScript Modules implementation. labels Feb 27, 2026
@Renegade334 Renegade334 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 Feb 27, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 1, 2026
@nodejs-github-bot nodejs-github-bot merged commit 0994958 into nodejs:main Mar 1, 2026
33 of 34 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 0994958

@Renegade334 Renegade334 deleted the doc-esm-detect-algo branch March 1, 2026 14:14
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. esm Issues and PRs related to the ECMAScript Modules implementation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants