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

Bump piscina to 4.3.1 #74

Merged
merged 3 commits into from
Dec 2, 2024
Merged

Bump piscina to 4.3.1 #74

merged 3 commits into from
Dec 2, 2024

Conversation

briankosw
Copy link
Contributor

While compiling a project using swc, I ran into the following issue.

 Error  The value of "val" is out of range. It must be >= 1 && <= 9007199254740991. Received 0

After digging through, I was able to figure out that the root cause is the same as piscinajs/piscina#491. Unfortunately I don't have a repro that I can share, but the issue is well-documented in the linked issue.

Copy link

changeset-bot bot commented Dec 2, 2024

🦋 Changeset detected

Latest commit: 033589e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@swc/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@CLAassistant
Copy link

CLAassistant commented Dec 2, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/piscina@4.3.0

View full report↗︎

@briankosw
Copy link
Contributor Author

@kdy1 do you mind taking a look at this? I might be missing some additional setup, hence the CI failure, but I'm confident the fix is right.

kdy1
kdy1 previously approved these changes Dec 2, 2024
@kdy1 kdy1 merged commit 591c6c8 into swc-project:main Dec 2, 2024
3 of 10 checks passed
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.

3 participants