Skip to content

Fix returned duration sign for non-ISO date arithmetic#618

Merged
Manishearth merged 1 commit into
boa-dev:mainfrom
Manishearth:non-iso-since-error
Nov 4, 2025
Merged

Fix returned duration sign for non-ISO date arithmetic#618
Manishearth merged 1 commit into
boa-dev:mainfrom
Manishearth:non-iso-since-error

Conversation

@Manishearth
Copy link
Copy Markdown
Contributor

@Manishearth Manishearth commented Nov 4, 2025

  'intl402/Temporal/PlainDateTime/prototype/since/wrapping-at-end-of-month-gregorian': [FAIL],
  'intl402/Temporal/PlainDateTime/prototype/until/wrapping-at-end-of-month-gregorian': [FAIL],

Surprised this didn't break more things.

Fixes #617

@nekevss
Copy link
Copy Markdown
Member

nekevss commented Nov 4, 2025

This should close #617, correct?

@Manishearth
Copy link
Copy Markdown
Contributor Author

Yes!

@Manishearth Manishearth changed the title Fix test failure around non-ISO date arithmetic Fix returned duration sign for non-ISO date arithmetic Nov 4, 2025
@Manishearth Manishearth merged commit c783303 into boa-dev:main Nov 4, 2025
8 checks passed
@Manishearth Manishearth deleted the non-iso-since-error branch November 4, 2025 19:57
aarongable pushed a commit to chromium/chromium that referenced this pull request Nov 4, 2025
Uplifts boa-dev/temporal#618,
boa-dev/temporal#619 to fix the remaining
test262 failures.

Bug: 401065166
Change-Id: Ie94eb956b4a0327d9e91c7ca43774cc4ecf3bf18
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7120501
Commit-Queue: Manish Goregaokar <manishearth@google.com>
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1540332}
jmajnert pushed a commit to jmajnert/chromium that referenced this pull request Nov 6, 2025
Uplifts boa-dev/temporal#618,
boa-dev/temporal#619 to fix the remaining
test262 failures.

Bug: 401065166
Change-Id: Ie94eb956b4a0327d9e91c7ca43774cc4ecf3bf18
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7120501
Commit-Queue: Manish Goregaokar <manishearth@google.com>
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1540332}
mohd-akram pushed a commit to gsource-mirror/chromium-src-third_party-rust that referenced this pull request Jan 9, 2026
Uplifts boa-dev/temporal#618,
boa-dev/temporal#619 to fix the remaining
test262 failures.

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

Calendar::date_until returns wrong results for negative durations

2 participants