Skip to content

YearMonth addition: Use ISO parameters, not calendared parameters, when subtracting#645

Merged
nekevss merged 1 commit intoboa-dev:mainfrom
Manishearth:pym-subtract
Dec 9, 2025
Merged

YearMonth addition: Use ISO parameters, not calendared parameters, when subtracting#645
nekevss merged 1 commit intoboa-dev:mainfrom
Manishearth:pym-subtract

Conversation

@Manishearth
Copy link
Copy Markdown
Contributor

The spec wants us to do this in calendar space

Fixes #643

@Manishearth Manishearth requested a review from nekevss December 8, 2025 23:11
…en subtracting

The spec wants us to do this in calendar space
aarongable pushed a commit to chromium/chromium that referenced this pull request Dec 9, 2025
Uplift boa-dev/temporal#645

Bug: 401065166
Change-Id: I19e4ef76a5b9f1f44279682ba42d085f42ec5832
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7238054
Auto-Submit: Manish Goregaokar <manishearth@google.com>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Commit-Queue: Manish Goregaokar <manishearth@google.com>
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1555809}
@nekevss nekevss merged commit 96d2657 into boa-dev:main Dec 9, 2025
8 checks passed
@Manishearth Manishearth deleted the pym-subtract branch December 11, 2025 00:09
mohd-akram pushed a commit to gsource-mirror/chromium-src-third_party-rust that referenced this pull request Jan 9, 2026
Uplift boa-dev/temporal#645

Bug: 401065166
Change-Id: I19e4ef76a5b9f1f44279682ba42d085f42ec5832
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7238054
Auto-Submit: Manish Goregaokar <manishearth@google.com>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Commit-Queue: Manish Goregaokar <manishearth@google.com>
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1555809}
NOKEYCHECK=True
GitOrigin-RevId: 392665fe043ac4ce33d44d6a7acdd3a425d5ec28
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.

PlainYearMonth arithmetic seems broken

2 participants