Skip to content

support skipLazyLoaded option#21

Merged
XhmikosR merged 3 commits intodependents:mainfrom
srsudar:lazy-requires
Feb 15, 2026
Merged

support skipLazyLoaded option#21
XhmikosR merged 3 commits intodependents:mainfrom
srsudar:lazy-requires

Conversation

@srsudar
Copy link
Copy Markdown
Contributor

@srsudar srsudar commented Dec 2, 2024

Overview

The node-detective-amd module has a skipLazyLoaded option that makes it possible to ignore dynamic requires.

This PR adds the same option to this library, making it possible to indicate that something like this is legal:

function foo() {
  const doThing = require('./a');
  doThing();
}

@XhmikosR XhmikosR merged commit 682353f into dependents:main Feb 15, 2026
10 checks passed
@XhmikosR XhmikosR changed the title support skipLazyLoaded option support skipLazyLoaded option Feb 15, 2026
@XhmikosR
Copy link
Copy Markdown
Member

@srsudar please make a PR to update README.md to mention the options and CC me, thanks.

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