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

Website prematurely marks Node.js 22 release as LTS #7153

Open
richardlau opened this issue Oct 29, 2024 · 8 comments
Open

Website prematurely marks Node.js 22 release as LTS #7153

richardlau opened this issue Oct 29, 2024 · 8 comments
Labels

Comments

@richardlau
Copy link
Member

URL:

https://nodejs.org/en

Browser Name:

Edge

Browser Version:

130.0.2849.56 (Official build) (64-bit)

Operating System:

Windows 11 Enterprise Version 22H2 build 22621.4169

How to reproduce the issue:

From earlier today, https://nodejs.org/en was incorrectly advertising Node.js 22.10.0 as Long Term Support (LTS):

image

This was incorrect -- the first LTS release of Node.js 22 is 22.11.0 (which I've just pushed up in the last couple of hours). I imagine this is some incorrect correlation between the release schedule and current date.

FWIW https://nodejs.org/dist/index.json / https://nodejs.org/dist/index.tab correctly indicate "lts": false for Node.js 22.10.0 and "lts": "Jod" for Node.js 22.11.0.

@richardlau richardlau added the bug label Oct 29, 2024
@AugustinMauroy
Copy link
Contributor

It's an error in logic: we base our calculations on the status, which is calculated from the start and end dates, so the release or not.

@ovflowd
Copy link
Member

ovflowd commented Oct 29, 2024

I believe the root issue is within https://github.com/cutenode/nodevu cc @bnb

@richardlau
Copy link
Member Author

I think @AugustinMauroy is correct, looks like code in this repo which is checking dates, and not the actual release metadata:

if (ltsStart && now >= new Date(ltsStart)) {
return 'LTS';
}

@ovflowd
Copy link
Member

ovflowd commented Oct 29, 2024

I think @AugustinMauroy is correct, looks like code in this repo which is checking dates, and not the actual release metadata:

if (ltsStart && now >= new Date(ltsStart)) {
return 'LTS';
}

Makes sense, and then we get the first item with LTS marked.

So the issue is that this was triggered before 22.11 actually went out but on the day that supposedly is valid, right?

@ovflowd
Copy link
Member

ovflowd commented Oct 29, 2024

Good thing is that since the data gets updated every 10 minutes (I think?), the links are correct already.

@richardlau
Copy link
Member Author

So the issue is that this was triggered before 22.11 actually went out but on the day that supposedly is valid, right?

Correct.

@richardlau
Copy link
Member Author

I believe the root issue is within https://github.com/cutenode/nodevu cc @bnb

There might also be a bug in nodevu as it appears that might also be doing time-based checks, but at least for this repository it would make sense to start with #7153 (comment).

@aduh95
Copy link
Contributor

aduh95 commented Oct 30, 2024

IMO it makes sense to recommend downloading Node.js 22.x on the day it becomes LTS, most users won't actually care what the metadata says, and it's not like 22.11.0 contained any feature not in 22.10.0.

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

No branches or pull requests

4 participants