Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

doc: update for solution to dual package hazard#53873

Open
taniishkaaa wants to merge 2 commits intonodejs:mainnodejs/node:mainfrom
taniishkaaa:issue-52174taniishkaaa/node:issue-52174Copy head branch name to clipboard
Open

doc: update for solution to dual package hazard#53873
taniishkaaa wants to merge 2 commits intonodejs:mainnodejs/node:mainfrom
taniishkaaa:issue-52174taniishkaaa/node:issue-52174Copy head branch name to clipboard

Conversation

@taniishkaaa
Copy link

updates doc/api/packages.md to recommend node/default conditions instead of require/import as a solution to the dual package hazard

Fixes: #52174

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/loaders

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 16, 2024
@aduh95
Copy link
Contributor

aduh95 commented Jul 16, 2024

Can you run make format-md to fix the linter errors please?

@taniishkaaa
Copy link
Author

Can you run make format-md to fix the linter errors please?

I'm facing some errors while running the command, I'll get it done as soon as i can

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recommend node/default conditions instead of require/import as a solution to the dual package hazard

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.