Skip to content

Revisions for Reiwa era #11611

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 3, 2019
Merged

Revisions for Reiwa era #11611

merged 3 commits into from
Apr 3, 2019

Conversation

rpetrusha
Copy link
Contributor

Revisions for Reiwa era

Describe your changes here.

Fixes #11610

@tarekgh

@rpetrusha rpetrusha added this to the April 2019 milestone Apr 1, 2019
@rpetrusha rpetrusha self-assigned this Apr 1, 2019
@tarekgh
Copy link
Member

tarekgh commented Apr 1, 2019

@rpetrusha the changes here looks good to me. I have a couple of questions/points:

  • Do we have any place in the docs that list the Japanese era names? would be nice to list the new name there.
  • Is it possible we can update the .NET blog too? I believe the information would be very useful there.

Thanks

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but left a few comments for you to consider

@@ -199,7 +202,7 @@ In specifying a era to the <xref:System.Globalization.Calendar.ToDateTime(System

### Calendars, eras, and date ranges: Relaxed range checks

Very much like individual calendars have supported date ranges, eras in the <xref:System.Globalization.JapaneseCalendar> and <xref:System.Globalization.JapaneseLunisolarCalendar> classes also have supported ranges. Previously, .NET used strict era range checks to ensure that a era-specific date was within the range of that era. An out-of-range date would throw a the .NET Framework uses relaxed ranged checking by default. That is, if a date is outside of the range of the specified era, the method throws an <xref:System.ArgumentOutOfRangeException>. Updates to all versions of the .NET Framework introduced relaxed era range checks; the attempt to instantiate an era-specific date that is outside the range of the specified era "overflow" into the following era, and no exception is thrown.
Very much like individual calendars have supported date ranges, eras in the <xref:System.Globalization.JapaneseCalendar> and <xref:System.Globalization.JapaneseLunisolarCalendar> classes also have supported ranges. Previously, .NET used strict era range checks to ensure that an era-specific date was within the range of that era. That is, if a date is outside of the range of the specified era, the method throws an <xref:System.ArgumentOutOfRangeException>. Currently, the .NET Framework uses relaxed ranged checking by default. Updates to all versions of the .NET Framework introduced relaxed era range checks; the attempt to instantiate an era-specific date that is outside the range of the specified era "overflows" into the following era, and no exception is thrown.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you mention Framework here, but what about Core?

@rpetrusha
Copy link
Contributor Author

@tarekgh, I'll update the blog either later today or first thing tomorrow morning. Also, the list of era names is in the documentation for the JapaneseCalendar and JapaneseLunisolarCalendar classes; I've just opened dotnet/dotnet-api-docs#2224 to revise the documentation for the Reiwa era.

mairaw and others added 2 commits April 1, 2019 16:35
Accepted Maira's suggestions.

Co-Authored-By: rpetrusha <[email protected]>
@rpetrusha
Copy link
Contributor Author

Closing and reopening to begin new build.

@rpetrusha rpetrusha closed this Apr 3, 2019
@rpetrusha rpetrusha reopened this Apr 3, 2019
@rpetrusha rpetrusha merged commit 8133044 into dotnet:master Apr 3, 2019
@rpetrusha rpetrusha deleted the reiwa-era branch April 3, 2019 21:36
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