Skip to content

feat(ecmascript): PlainTime.prototype.toJSON#984

Open
jesperkha wants to merge 2 commits intotrynova:mainfrom
jesperkha:temporal/plaintime-toJson
Open

feat(ecmascript): PlainTime.prototype.toJSON#984
jesperkha wants to merge 2 commits intotrynova:mainfrom
jesperkha:temporal/plaintime-toJson

Conversation

@jesperkha
Copy link
Copy Markdown
Contributor

No description provided.

@jesperkha jesperkha changed the title feat(ecmascript): Temporal.PlainTime.prototype.toJSON feat(ecmascript): PlainTime.prototype.toJSON Apr 19, 2026
@jesperkha jesperkha force-pushed the temporal/plaintime-toJson branch from 861dafa to ab7ac1d Compare April 19, 2026 13:40
Comment thread nova_vm/src/ecmascript/builtins/temporal/plain_time/plain_time_prototype.rs Outdated
aapoalas
aapoalas previously approved these changes Apr 24, 2026
Copy link
Copy Markdown
Member

@aapoalas aapoalas left a comment

Choose a reason for hiding this comment

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

LGTM but fixing the nitpick would maybe be a nice cleanup: you can get rid of all the unbinds as well that way.

@aapoalas aapoalas force-pushed the temporal/plaintime-toJson branch from ab7ac1d to ecb6fe5 Compare April 24, 2026 13:50
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