-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Upgrade Typescript to 4.3.5 #104470
Upgrade Typescript to 4.3.5 #104470
Changes from 1 commit
67b2548
c49094d
7892e6f
29044a3
5c0d31c
9d69f94
3837610
5d384f5
e2c53d5
9e8e7df
b041686
801ce46
ce3215c
2c9e422
c7f19a4
924c0c0
b882621
9b6f3c4
c569d3e
a6c8e07
d4bc9e0
930bf97
720b950
8bf1ced
949bd50
5f857c0
c28fbe4
297c1bb
3a6f7f6
693383a
27fd24d
c6b5475
d23dd2f
aa2ef4a
bd02f80
dcea043
be6f865
e8dcba7
93dc843
7cd8af6
1e54200
355d9e6
7f8adc3
33574fc
8ac8edf
05c4ce8
85994f4
a5c7f7a
6e7834f
7059a11
4fa7f9d
2a2d248
57c0d2b
cc86f85
71798c8
3aa2fcb
24897fa
4982f40
a200694
4174720
809f798
d35528a
17bace0
e333fdf
fdced85
a7ceb1c
d634294
0e8b4a2
74d17e4
e8b287b
919f1f3
8c07018
8db01ef
66a566e
8480502
ce0ea5f
c002a41
9513d13
ae76664
c6dcf50
bce6445
b1756f3
aa34a9a
20a1822
8d868e8
638be92
a6098e6
805d96f
5f9f214
375d66f
5f45a24
9235871
6259e15
16fe24b
4a30b12
cf8fa9d
25dda80
a9b78e9
49b2590
1c76872
bfc7d14
dd5e777
cc52b77
f2ba37f
f54f6b1
44cb2b3
dc24afd
3d87631
c01bfab
2486436
921bfc1
47068e2
ae216ef
65adb76
cb0f129
417d703
c82a4de
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
|
@@ -37,17 +37,15 @@ export type KnownHeaders = KnownKeys<IncomingHttpHeaders>; | |||
* Http request headers to read. | ||||
* @public | ||||
*/ | ||||
export type Headers = { [header in KnownHeaders]?: string | string[] | undefined } & { | ||||
export interface Headers { | ||||
[header: string]: string | string[] | undefined; | ||||
}; | ||||
} | ||||
|
||||
/** | ||||
* Http response headers to set. | ||||
* @public | ||||
*/ | ||||
export type ResponseHeaders = | ||||
| Record<KnownHeaders, string | string[]> | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ran into the same issue in Fleet. Just replacing with a
|
||||
| Record<string, string | string[]>; | ||||
export type ResponseHeaders = Record<string, string | string[]>; | ||||
|
||||
const normalizeHeaderField = (field: string) => field.trim().toLowerCase(); | ||||
|
||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does
tsc
fail if you put the old definition ofHeaders
back?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it can be restored - pushed 47068e2