Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

docs: Add global install instructions, change phrasing #2037

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RichardLitt
Copy link

I have added the way to globally install node-sass from the CLI,
which isn't present in the README otherwise. I have also removed a
sentence which says that node-sass is simplistic. In both cases,
it is assumed that someone using the program knows how to use npm
to install command line packages, and that the user has some idea
about the level of complexity usual to an npm package. Both of
these assumptions are assumptions - my wording makes the package
more friendly to beginners by not assuming they know this.

tl;dr: just a small change to help new users.

I have added the way to globally install node-sass from the CLI,
which isn't present in the README otherwise. I have also removed a
sentence which says that node-sass is simplistic. In both cases,
it is assumed that someone using the program knows how to use npm
to install command line packages, and that the user has some idea
about the level of complexity usual to an npm package. Both of
these assumptions are assumptions - my wording makes the package
more friendly to beginners by not assuming they know this.

tl;dr: just a small change to help new users.
@xzyfer
Copy link
Contributor

xzyfer commented Jul 4, 2017

my wording makes the package more friendly to beginners by not assuming they know this

I whole heartedly support this sentiment. However we do not recommend, or support install node-sass globally.

@RichardLitt
Copy link
Author

How would the CLI client work, otherwise?

@xzyfer
Copy link
Contributor

xzyfer commented Jul 4, 2017

The primary usecase for the CLI is in npm scripts. Alternatively you can use something like npx.

However global installs of node-sass have been notoriously troublesome and are not something we'd like to encourage.

@RichardLitt
Copy link
Author

How about the language I just pushed?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants