Gozala / querystring

Node's querystring module for all engines.
MIT License
287 stars 72 forks source link

Add “Not npm install query-string !!!!!” to the readme! #51

Closed yinxulai closed 3 years ago

yinxulai commented 3 years ago

ref:https://github.com/sindresorhus/query-string/blob/main/readme.md#install

image

yinxulai commented 3 years ago

@Gozala

yinxulai commented 3 years ago

I thought it was an interesting interaction.

medikoo commented 3 years ago

@yinxulai I think it's best to keep documentation minimal. We may add a lot of potentially valuable hints, but if they do not help anyone, they make reading documentation just harder, as it's just noise user has to get through.

In this specific case, I think users do not have a problem installing correct package, so I don't think there's a need to add such note.

yinxulai commented 3 years ago

That's good.