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.

Sign up for GitHub

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

Jump to bottom

[CI][Release][JS] Node.js 16 is old for verification #39020

Closed
kou opened this issue Dec 1, 2023 · 0 comments · Fixed by #39021
Closed

[CI][Release][JS] Node.js 16 is old for verification #39020

kou opened this issue Dec 1, 2023 · 0 comments · Fixed by #39021
Assignees
@kou
Labels
Component: JavaScript Component: Release Type: enhancement
Milestone
15.0.0

Comments

@kou
Copy link
Member

kou commented Dec 1, 2023

Describe the enhancement requested

Node.js 16 reached EOL: https://nodejs.org/en/about/previous-releases

Our release verification CI job is also started failing:

https://github.com/ursacomputing/crossbow/actions/runs/7005228039/job/19054548400#step:7:39

error rollup@4.3.0: The engine "node" is incompatible with this module. Expected version ">=18.0.0". Got "16.20.2"

Component(s)

JavaScript, Release
kou added a commit to kou/arrow that referenced this issue Dec 1, 2023
@kou
apacheGH-39020: [CI][Release][JS] Use Node.js 18 instead of 16
7ae4d4f
@github-actions github-actions bot mentioned this issue Dec 1, 2023
Merged
kou added a commit to kou/arrow that referenced this issue Dec 1, 2023
@kou
apacheGH-39020: [CI][Release][JS] Use Node.js 18 instead of 16
71ca928
kou added a commit to kou/arrow that referenced this issue Dec 1, 2023
@kou
apacheGH-39020: [CI][Release][JS] Use Node.js 18 instead of 16
8b975e2
kou added a commit to kou/arrow that referenced this issue Dec 1, 2023
@kou
apacheGH-39020: [CI][Release][JS] Use Node.js 18 instead of 16
689f851
@kou kou mentioned this issue Dec 1, 2023
Merged
kou added a commit to kou/arrow that referenced this issue Dec 1, 2023
@kou
apacheGH-39020: [CI][Release][JS] Use Node.js 18 instead of 16
6cfe869
@kou kou closed this as completed in #39021 Dec 1, 2023
kou added a commit that referenced this issue Dec 1, 2023
@kou
GH-39020: [CI][Release][JS] Use Node.js 18 instead of 16 (#39021)
bcd096d 
### Rationale for this change

Node.js 16 reached EOL.

### What changes are included in this PR?

Use Node.js 18. It's the oldest maintained LTS.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: #39020

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 15.0.0 milestone Dec 1, 2023
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
@kou @dgreiss
apacheGH-39020: [CI][Release][JS] Use Node.js 18 instead of 16 (apach…
ff3e8c1 
…e#39021)

### Rationale for this change

Node.js 16 reached EOL.

### What changes are included in this PR?

Use Node.js 18. It's the oldest maintained LTS.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: apache#39020

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kou kou

Labels
Component: JavaScript Component: Release Type: enhancement
Projects
None yet
Milestone
15.0.0
Development

Successfully merging a pull request may close this issue.

GH-39020: [CI][Release][JS] Use Node.js 18 instead of 16 kou/arrow
1 participant
@kou