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

Document supported node version #280

Merged
merged 2 commits into from
Apr 27, 2016
Merged

Document supported node version #280

merged 2 commits into from
Apr 27, 2016

Conversation

xPaw
Copy link
Member

@xPaw xPaw commented Apr 26, 2016

Fixes #231.

engines field only affects new installations, and only emits a warning like this:

npm WARN engine thelounge@1.5.0: wanted: {"node":">=8.2.0"} (current: {"node":"5.4.1","npm":"3.3.12"})

It does not block installation. I documented 4.2.0 because it's the lowest LTS version, as seen here: https://nodejs.org/en/download/releases/

It only took about a month to release 4.2.0 after 4.0.0, so it's not going to be a big problem.

@xPaw xPaw added the Type: Documentation Lack of documentation, improvement suggestion, or PRs that address these. label Apr 26, 2016
- '5.6'
- '4.3'
- '0.12'
- 'node'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uses latest stable

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd rather see more specific versions, like latest 4.x, 5.x and 6.x, no?

@maxpoulin64
Copy link
Member

It does not block installation.

Perfect. Not limiting for us, but people are warned it's not tested/supported yet they still can run it if they really want. 👍

@@ -35,6 +35,12 @@ the review process quicker and more streamlined.

## Install

To use The Lounge you must have [Node.js](https://nodejs.org/en/download/) installed.
The oldest Node.js version we support is 4.2.0.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happened before 4.2.0 (and > 0.12) so that we don't support them?
I think that in the future, we could get away with supporting current stable and LTS proposed by the Node.js website, what do you think?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

io.js happened. 4.0 was next release after 0.12. The issue discusses why we should drop the support for 0.x.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I meant <= 4.2 and > 4.0. Why supporting 4.2 as oldest version and not 4.0 is a better question, actually.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It only took about a month to release 4.2.0 after 4.0.0.

4.2.0 and up is the LTS.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where did you get this from? 4.x is the LTS branch, and the newest version of it is the most up to date version of it. Being LTS means that they will provide security updates to it, and bug fixes. That means that each indibidual point release is not "LTS", but the branch itself is LTS.

We aren't going to update this every time there's a new patch to 4.x, so just stick it to 4.0

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See https://nodejs.org/en/download/releases/ and https://github.com/nodejs/node/blob/v4.2.0/CHANGELOG.md

Version 4.2.0 'Argon'
The first Node.js LTS release! See https://github.com/nodejs/LTS/ for details of the LTS process.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's just because the April releases aren't LTS until 6 months after release (to allow for the small bugs to be eeked out).

Copy link
Member Author

@xPaw xPaw Apr 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4.4.3 released on 2016-04-12 is already LTS though.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not what I meant. The major releases, not any release in April.

I think you misunderstand how LTS works. The point of LTS is that the release branch will be updated with bug fixes and security updates.

4.4.3 isn't LTS. That doesn't mean anything. 4.4.3 is, and always will be, exactly the same code. It will never change. So how can there be any support for it?

What is LTS is the 4.x release branch. They word it weirdly in node, but the point is that 4.x will get bug fixes and security updates until it is no longer LTS (next April).

When a major April release happens (4.x was different in that it was released in September, but from now on all the LTS releases will be from April major releases), 6 months later it becomes LTS, and is the stablest of the stable that they have, and will be updated with bug fixes and security updates for 18 months.

So, it doesn't mean anything to say "4.2.0 is LTS" or "4.4.3 is LTS". No, 4.x is LTS. That's all.

@AlMcKinlay
Copy link
Member

I'm happy with this now. I'm not sure there's an advantage to saying >=4.2.0, but I'm not going to argue too mcuh.

I won't merge this, because @astorije didn't sound very happy with us giving an error, so I'll let him decide if he cares enough about it :-)

@astorije
Copy link
Member

I won't merge this, because @astorije didn't sound very happy with us giving an error, so I'll let him decide if he cares enough about it :-)

Not about the error itself but with the fact that we have one more dependency just for that (at our level, regardless of what dependencies themselves have in their dependencies.

That being said, everyone is 👍 so I am too, and hopefully we'll get to remove that later :-)

@astorije astorije merged commit 177041c into master Apr 27, 2016
@astorije astorije deleted the min-node-ver branch April 27, 2016 14:03
@astorije astorije added this to the ★ Next Release milestone May 15, 2016
matburnham pushed a commit to matburnham/lounge that referenced this pull request Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Lack of documentation, improvement suggestion, or PRs that address these.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants