Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(node): mark ws as external module #1716

Merged
merged 2 commits into from
Apr 8, 2024
Merged

build(node): mark ws as external module #1716

merged 2 commits into from
Apr 8, 2024

Conversation

owenpearson
Copy link
Member

@owenpearson owenpearson commented Mar 27, 2024

fixes an issue with realtime connections using the bun runtime

(also marks got as external just so we aren't pinned to a specific version)

@owenpearson owenpearson requested a review from VeskeR March 27, 2024 12:40
@github-actions github-actions bot temporarily deployed to staging/pull/1716/features March 27, 2024 12:40 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/bundle-report March 27, 2024 12:41 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/typedoc March 27, 2024 12:41 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/features March 27, 2024 12:53 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/bundle-report March 27, 2024 12:53 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/typedoc March 27, 2024 12:54 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/features March 27, 2024 15:39 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/bundle-report March 27, 2024 15:39 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1716/typedoc March 27, 2024 15:40 Inactive
grunt/esbuild/build.js Outdated Show resolved Hide resolved
Copy link
Contributor

@VeskeR VeskeR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@VeskeR
Copy link
Contributor

VeskeR commented Apr 8, 2024

Merging this PR on Owen's behalf, as he is out of the office, and we want to release this fix as soon as possible.

@VeskeR VeskeR merged commit f042f86 into main Apr 8, 2024
12 checks passed
@VeskeR VeskeR deleted the node-ws-external branch April 8, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants