Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Intl 2021 Updates#45647

Merged
orta merged 15 commits into
microsoft:mainmicrosoft/TypeScript:mainfrom
orta:ort-intl-updatesorta/TypeScript:ort-intl-updatesCopy head branch name to clipboard
Sep 8, 2021
Merged

Intl 2021 Updates#45647
orta merged 15 commits into
microsoft:mainmicrosoft/TypeScript:mainfrom
orta:ort-intl-updatesorta/TypeScript:ort-intl-updatesCopy head branch name to clipboard

Conversation

@orta

@orta orta commented Aug 30, 2021

Copy link
Copy Markdown
Contributor

Re: #45646 - no breaking changes in here though

This is quite a few PRs + issues combined, rebased etc:

I've tried to reduce the overall character count for some of the docs (e.g. removing some JSDocs which are redundant because the type system will tell you the types of things) and removed all the specs links, I can't imagine your average Intl API consumer wanting to read the spec (over MDN).

Also adds some fresh docs about how lib files work to the README, covering how Intl is a wee bit special in comparison to some of the other cases we have in the .d.ts files.

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Aug 30, 2021
@orta

orta commented Aug 31, 2021

Copy link
Copy Markdown
Contributor Author

@typescript-bot run dt

@typescript-bot

typescript-bot commented Aug 31, 2021

Copy link
Copy Markdown
Contributor

Heya @orta, I've started to run the parallelized Definitely Typed test suite on this PR at 5f67f65. You can monitor the build here.

@orta

orta commented Aug 31, 2021

Copy link
Copy Markdown
Contributor Author

I've double checked all the es5 Intl properties, and they are in the right place (they were spec'd in 2012 which is a reasonable approximation IMO)

@orta

orta commented Aug 31, 2021

Copy link
Copy Markdown
Contributor Author

@longlho any chance I could get your eyes over this? (and sorry it's taken a while!)

@longlho

longlho commented Aug 31, 2021

Copy link
Copy Markdown
Contributor

Yup will take a look asap

@sandersn sandersn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a lot of minor edit suggestions.

Comment thread src/lib/README.md
Comment thread src/lib/README.md Outdated
Comment thread src/lib/es2018.intl.d.ts
Comment thread src/lib/README.md Outdated
Comment thread src/lib/README.md Outdated
Comment thread src/lib/README.md Outdated
Comment thread src/lib/README.md Outdated
Comment thread src/lib/es2020.intl.d.ts Outdated
Comment on lines 220 to 223
hourCycle?: "h11" | "h12" | "h23" | "h24" | undefined;
}

interface DateTimeFormatOptions {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Probably should merge these two DateTimeFormatOptions declarations

Comment thread src/lib/es2020.intl.d.ts

@longlho longlho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall I think es2021 is missing a ton of stuff but I think it's a good enough set of changes in the scaffolding for others to fill in.

Comment thread src/lib/es2018.intl.d.ts
Comment thread src/lib/es2018.intl.d.ts Outdated
Comment thread src/lib/es2021.intl.d.ts
Comment thread src/lib/es2021.intl.d.ts
orta and others added 3 commits September 6, 2021 15:48
Co-authored-by: Nathan Shively-Sanders <293473+sandersn@users.noreply.github.com>
@orta

orta commented Sep 6, 2021

Copy link
Copy Markdown
Contributor Author

OK, thanks for the feedback everyone - assuming this is green I will merge weds and start closing all the linked PRs

Comment thread src/lib/es2021.intl.d.ts Outdated
Comment thread src/lib/es2020.intl.d.ts Outdated
@longlho

longlho commented Sep 6, 2021

Copy link
Copy Markdown
Contributor

Editorial suggestions but overall lgtm

@orta

orta commented Sep 8, 2021

Copy link
Copy Markdown
Contributor Author

Thanks folks

@orta orta merged commit 07fd7bc into microsoft:main Sep 8, 2021
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.