Fix IntlExtension ignoring explicit formats when a date formatter prototype is set#4859
Conversation
|
📋 PR Summary : I'll produce the JSON. Findet me be careful and just output the finding for the calendar and general notes.-Let me write valid JSON.-The main finding: calendar sibling using |
There was a problem hiding this comment.
📋 Upsun Dispatch Review: full · 3 files reviewed · 🟡 1 warning
Review details
Commit: Commit 9eb3a9f
Model: claude-opus-4-8
Panel: correctness · design
9eb3a9f to
5100643
Compare
There was a problem hiding this comment.
📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found · 1 still open
Review details
Commit: Commit 5100643
Model: claude-opus-4-8
Panel: correctness · design
Outstanding from earlier reviews:
- #3566173108 — extra/intl-extra/IntlExtension.php:487: Silently drops a prototype's custom pattern for format_date/format_time, a behavior change not covered by tests.
5100643 to
cc6bda8
Compare
There was a problem hiding this comment.
📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found · 1 still open
Review details
Commit: Commit cc6bda8
Model: claude-opus-4-8
Panel: correctness · design
Outstanding from earlier reviews:
- #3566173108 — extra/intl-extra/IntlExtension.php:487: Silently drops a prototype's custom pattern for format_date/format_time, a behavior change not covered by tests.
cc6bda8 to
3c93cff
Compare
There was a problem hiding this comment.
📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found · 1 still open
Review details
Commit: Commit 3c93cff
Model: claude-opus-4-8
Panel: correctness · design
Outstanding from earlier reviews:
- #3566173108 — extra/intl-extra/IntlExtension.php:487: Silently drops a prototype's custom pattern for format_date/format_time, a behavior change not covered by tests.
91816ad to
083b6dc
Compare
There was a problem hiding this comment.
📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found
Review details
Commit: Commit 083b6dc
Model: claude-opus-4-8
Panel: correctness · robustness · design
Closes #3845