Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't mention npm command for installation #269

Closed
wants to merge 1 commit into from
Closed

Don't mention npm command for installation #269

wants to merge 1 commit into from

Conversation

SyntaxColoring
Copy link
Contributor

@SyntaxColoring SyntaxColoring commented Jan 23, 2023

The v4.3.1 release notes say:

Also note that the npm package manager is no longer officially supported by The Lounge and we now only support using yarn.

I take this to mean that we should always install with yarn global add, and never with npm install --global. That would mean that this mention of npm install --global on the "Install and upgrade" page is obsolete. So, remove it.

I'm unfamiliar with Javascript package management, so forgive me if I've missed the point!

@SyntaxColoring
Copy link
Contributor Author

There's another mention right below:

Note that installing from npm or yarn does not daemonize nor autostart The Lounge.

Not sure if it's better to leave that one in or take it out.

@brunnre8
Copy link
Member

7e936c2

@brunnre8 brunnre8 closed this Feb 13, 2023
@brunnre8
Copy link
Member

Thank you

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

Successfully merging this pull request may close these issues.

None yet

2 participants