Skip to content

Conversation

@Gumichocopengin8
Copy link
Contributor

@Gumichocopengin8 Gumichocopengin8 commented Oct 27, 2025

Update boa to v0.21.0. More conformance improvements and bug fixes.

https://github.com/boa-dev/boa/blob/main/CHANGELOG.md#v0210

@Gumichocopengin8 Gumichocopengin8 marked this pull request as ready for review October 27, 2025 13:13
@Gumichocopengin8
Copy link
Contributor Author

Looks like CI doesn't run probably because it's from fork.

Copy link
Contributor

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

a few breaking changes, but dont look too bad

@Gumichocopengin8
Copy link
Contributor Author

Gumichocopengin8 commented Oct 28, 2025

Rust-analyzer isn’t helping with type hints, so fixing the errors has been tricky.
I’ll close this PR for now in case someone else wants to take over the Boa update. Appreciate the review!

@Gumichocopengin8 Gumichocopengin8 deleted the chore/update-boa-0.21.0 branch October 28, 2025 00:39
@Gumichocopengin8 Gumichocopengin8 restored the chore/update-boa-0.21.0 branch October 31, 2025 09:44
@Gumichocopengin8
Copy link
Contributor Author

Gumichocopengin8 commented Oct 31, 2025

@mattsse I managed to make it work. Could you take a look at it again? Thanks.
Still not sure why rust-analyzer doesn't work though.

@mattsse
Copy link
Contributor

mattsse commented Oct 31, 2025

amazing

@Gumichocopengin8 Gumichocopengin8 changed the title chore: update boa to v0.21.0 chore: update Boa to v0.21.0 Oct 31, 2025
})
} else {
val.to_json(ctx)
val.to_json(ctx)?.ok_or_else(|| {
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

very nice, ty!

@mattsse mattsse merged commit ee2021d into paradigmxyz:main Oct 31, 2025
12 checks passed
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.

3 participants