search

samtgarson / nuxt-env

Inject env vars for your Nuxt app at runtime
170 stars 15 forks source link

Installation instructions should include npm #23

Open Soviut opened 4 years ago

Soviut commented 4 years ago

Only showing yarn add nuxt-env in the README had one of our junior developers thinking that the module needed to be installed with Yarn, while the rest of our code base uses npm.

I tried forking and creating a PR but can't seem to target your master branch. So here it is https://github.com/Soviut/nuxt-env/pull/1

samtgarson commented 4 years ago

It's pretty standard practice to give one example in a README—your junior developer would only make that mistake once.

Others have been able to open PRs, if you want to give that another try I'd be happy to merge it.

Soviut commented 4 years ago

I wouldn't consider the installation instructions an example. Most packages I've seen show both.

I'll see about retrying the PR.