Skip to content

feat(ecmascript): get PlainTime.prototype.minute#955

Merged
aapoalas merged 4 commits intotrynova:mainfrom
jesperkha:minutes
Feb 24, 2026
Merged

feat(ecmascript): get PlainTime.prototype.minute#955
aapoalas merged 4 commits intotrynova:mainfrom
jesperkha:minutes

Conversation

@0Zeno
Copy link
Contributor

@0Zeno 0Zeno commented Feb 21, 2026

No description provided.

@0Zeno 0Zeno changed the title Minutes feat(ecmascript): get PlainTime.prototype.minute Feb 21, 2026
Copy link
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.

Needs rebase, and also most likely will need to update test262 results.

See CONTRIBUTING.md for how to run the test262 update (requires checking out the test262 git submodule and running the test262 cargo bin command with the -u update flag).

Copy link
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.

Still needs cargo fmt and the test results to be updated. This should do the trick, I believe:

git submodule init && git submodule update && cargo build --profile dev-fast && cargo run --bin test262 --profile dev-fast -- -u

Copy link
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, great work! <3

@aapoalas aapoalas merged commit c9616fd into trynova:main Feb 24, 2026
8 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