Skip to content

Conversation

tysonclugg
Copy link

Avoids the following error when running pnpm run test:

test browser
ℹ Browser "chromium" setup complete.
✘ [ERROR] Could not resolve "events"

    dist/test/utils/create-pubsub.js:1:32:
      1 │ import { setMaxListeners } from 'events';
        │                                 ~~~~~~~~
        ╵                                 "./events"

  The package "events" wasn't found on the file system but is built into node. Are you trying to
  bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

Also allows errors not related to failed imports to propagate correctly.

@tysonclugg tysonclugg requested a review from a team as a code owner April 16, 2025 01:30
@tysonclugg tysonclugg force-pushed the fix/import_node_events branch from 21a99e1 to 670784e Compare August 17, 2025 22:53
Avoids the following error when running `pnpm run test`:
```
test browser
ℹ Browser "chromium" setup complete.
✘ [ERROR] Could not resolve "events"

    dist/test/utils/create-pubsub.js:1:32:
      1 │ import { setMaxListeners } from 'events';
        │                                 ~~~~~~~~
        ╵                                 "./events"

  The package "events" wasn't found on the file system but is built into node. Are you trying to
  bundle for node? You can use "platform: 'node'" to do that, which will remove this error.
```
@tysonclugg tysonclugg force-pushed the fix/import_node_events branch from 670784e to 7b5a4d2 Compare August 20, 2025 08:36
@tysonclugg
Copy link
Author

@achingbrain Now importing from @libp2p/interface as requested, ready for review.

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