Skip to content

Commit

Permalink
Adjust setWeek to not use default argument value
Browse files Browse the repository at this point in the history
  • Loading branch information
@kossnocorp
kossnocorp committed Jun 3, 2021
1 parent 121e95e commit ea3f050
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/setWeek/index.ts
View file
Expand Up @@ -55,7 +55,7 @@ import {
export default function setWeek(
dirtyDate: Date | number,
dirtyWeek: number,
options: LocaleOptions & WeekStartOptions & FirstWeekContainsDateOptions = {}
options: LocaleOptions & WeekStartOptions & FirstWeekContainsDateOptions

This comment has been minimized.

Sign in to view

Copy link
@fturmel

fturmel Jun 29, 2021

Member

@kossnocorp I think you still need options?: here to make it optional without initializing the value to an empty object. Lots of tsc errors in the test file because the wrong number of required arguments are being passed.
): Date {
requiredArgs(2, arguments)

Expand Down

0 comments on commit ea3f050

Please sign in to comment.