fix(types): add missing cache prop to RequestInit

[rindeal]

Also sort the props.

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

• I have read and agreed to the Developer's Certificate of Origin
• Tested
• Benchmarked (optional)
• Documented
• Review ready
• In review
• Merge ready

[eXhumer]

Lint issues, use 2 spaces instead of 4.

I don't think undici.fetch supports caching yet (See #2760).

If you want to implement some sort of in-memory caching, see #2760 (comment).

[rindeal]

Lint issues, use 2 spaces instead of 4.

yeah, will fix

I don't think undici.fetch supports caching yet (See #2760).

If you want to implement some sort of in-memory caching, see #2760 (comment).

The prop is supported and that's all that matters in this case

undici/lib/web/fetch/request.js

Lines 974 to 979 in dca0aa0

	{
	key: 'cache',
	converter: webidl.converters.DOMString,
	// https://fetch.spec.whatwg.org/#requestcache
	allowedValues: requestCache
	},

[mcollina]

Can you add some type tests for this? We use tsd.

[Uzlopak]

And tomorrow somebody argues, that because the cache field is in the types, we need to implement asap caching.

[rindeal]

Can you add some type tests for this? We use tsd.

✅

[Uzlopak]

@KhafraDev

[rindeal]

bump @mcollina

[mcollina]

lgtm