Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

chore: remove unnecessary stringification from node-fetch snippets #1004

Merged
merged 3 commits into from
Oct 21, 2020

Conversation

erunion
Copy link
Member

@erunion erunion commented Oct 21, 2020

🧰 What's being changed?

Upgrades @readme/httpsnippet to 2.2.3 to remove unnecessary .toString() calls when supplying URLSearchParams to node-fetch body` options.

node-fetch can accept both a string or the raw object, so since the snippet looks cleaner without the stringification we're removing it.

@erunion erunion added the scope:dependency Pull requests that update a dependency file label Oct 21, 2020
@erunion erunion merged commit 2218ed4 into next Oct 21, 2020
@erunion erunion deleted the chore/update-httpsnippet branch October 21, 2020 21:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope:dependency Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant