Skip to content

lib(esnext.intl): add missing minimalDays to WeekInfo interface#63591

Open
smedavarapu1 wants to merge 1 commit into
microsoft:mainfrom
smedavarapu1:fix/weekinfo-minimaldays
Open

lib(esnext.intl): add missing minimalDays to WeekInfo interface#63591
smedavarapu1 wants to merge 1 commit into
microsoft:mainfrom
smedavarapu1:fix/weekinfo-minimaldays

Conversation

@smedavarapu1

@smedavarapu1 smedavarapu1 commented Jun 26, 2026

Copy link
Copy Markdown

Summary

Fixes #61713

The WeekInfo interface in src/lib/esnext.intl.d.ts was missing the minimalDays property, which is part of the Intl Locale Info API (Stage 4).

The JSDoc comment on getWeekInfo() already documented all three properties (firstDay, weekend, and minimalDays), but the WeekInfo return type interface only declared firstDay and weekend. This change adds the missing minimalDays: number property with the appropriate JSDoc and MDN reference link.

Checklist

  • There is an associated issue in the Backlog milestone (Missing Intl.Locale.prototype.getWeekInfo() #61713)
  • Code is up-to-date with the main branch
  • You've successfully run hereby runtests locally
  • There are new or updated unit tests validating the change

Copilot AI review requested due to automatic review settings June 26, 2026 18:14
@github-project-automation github-project-automation Bot moved this to Not started in PR Backlog Jun 26, 2026
@typescript-automation typescript-automation Bot added the For Backlog Bug PRs that fix a backlog bug label Jun 26, 2026

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For Backlog Bug PRs that fix a backlog bug

Projects

Status: Not started

Development

Successfully merging this pull request may close these issues.

Missing Intl.Locale.prototype.getWeekInfo()

2 participants