search

netlify / netlify-credential-helper

Git credential helper to use Netlify's API as authentication backend
MIT License
17 stars 4 forks source link

fix(docs): remove plugin installation part from readme #55

Closed erezrokah closed 3 years ago

erezrokah commented 3 years ago

I changed netlify lm:setup to netlify lm:install as setup also provisions the Large Media service and requires the user to have a linked site.

I think the README for this repo should guide the user to only install the credential helper, WDYT?

keiko713 commented 3 years ago

I think the README for this repo should guide the user to only install the credential helper, WDYT?

yeah good point. but I also like how we were pointing the preferred way, as it can be confusing to see the connection of "what this helper is used for?". this helper is heavily specialized for LM and I would like to leave the mention somewhere for this.

erezrokah commented 3 years ago

yeah good point. but I also like how we were pointing the preferred way, as it can be confusing to see the connection of "what this helper is used for?". this helper is heavily specialized for LM and I would like to leave the mention somewhere for this.

I wasn't clear with my suggestion. I was referring to the change from netlify lm:setup to netlify lm:install, so I think we're on the same page here.