Skip to content

Conversation

@Rob-S
Copy link
Contributor

@Rob-S Rob-S commented Jul 31, 2025

"DateTimeOffset(Int32, Int32, Int32, Int32, Int32, Int32, Int32, Calendar, TimeSpan)" does not initialize the new instance using "offset of a specified calendar." Per elsewhere on the same page, the calendar is used to interpret the year, month and day, not the offset. The "calendar" is missing from the similar constructor which includes "microsecond".

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

"DateTimeOffset(Int32, Int32, Int32, Int32, Int32, Int32, Int32, Calendar, TimeSpan)" does not initialize the new instance using "offset of a specified calendar." Per elsewhere on the same page, the calendar is used to interpret the year, month and day, _not_ the offset.
The "calendar" is missing from the similar constructor which includes "microsecond".
@Rob-S Rob-S requested a review from a team as a code owner July 31, 2025 21:28
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jul 31, 2025
@gewarren gewarren enabled auto-merge (squash) December 23, 2025 02:18
@gewarren gewarren merged commit 018f366 into dotnet:main Dec 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.DateTime community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants