fisker Cheung
fisker Cheung
Once this gets reviewed, we can revert https://github.com/sindresorhus/eslint-plugin-unicorn/pull/2339/commits/9bc7b933cbb1a34be79f4163cca1e75bb271b68e before merge.
What I was saying, I think `queueMicrotask` should be preferred, but check `process.nextTick` is not going to help me at all.
Turns out `no-hex-escape` has the same issue.
These doesn't thows ``` fetch('/', {method: 'OPTIONS', body: 'foo'}) fetch('/', {method: 'TRACE', body: 'foo'}) fetch('/', {method: 'CONNECT', body: 'foo'}) ``` The spec only forbids `GET` and `HEAD`.
@sindresorhus Thoughts?
> the examples in the documentation feel to me like they are worse after the recommended change Why is that? Escaped `\\` is ugly to me.
Should not enforce `POST`, but forbid `'GET'`(or omitted)/`'HEAD'`
> No HTTP method strictly forbids a body The fetch spec forbid it, > If either init["body"] exists and is non-null or inputBody is non-null, and request’s method is `GET`...
`no-invalid-fetch-options`?
Sorry, we haven't decided yet, and the change should make after we merge `next` branch to `main`.