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

ci: Use NodeJS 16 #1986

Merged
merged 7 commits into from
Jan 28, 2022
Merged

ci: Use NodeJS 16 #1986

merged 7 commits into from
Jan 28, 2022

Conversation

tolusha
Copy link
Collaborator

@tolusha tolusha commented Jan 27, 2022

Signed-off-by: Anatolii Bazko abazko@redhat.com

What does this PR do?

  • Use NodeJS 16
  • Unify PRs checks

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

eclipse-che/che#21075

How to test this PR?

N/A

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@benoitf
Copy link
Collaborator

benoitf commented Jan 27, 2022

probably

"@types/node": "^12",
and
"node": ">=12.0.0"
could be bumped as well

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Copy link
Collaborator

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

approving overall changes, but it seems there are still compilation failures

@openshift-ci
Copy link

openshift-ci bot commented Jan 27, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: benoitf, tolusha

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@openshift-ci openshift-ci bot removed the lgtm label Jan 27, 2022
@openshift-ci
Copy link

openshift-ci bot commented Jan 27, 2022

New changes are detected. LGTM label has been removed.

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@tolusha
Copy link
Collaborator Author

tolusha commented Jan 27, 2022

/retest

1 similar comment
@benoitf
Copy link
Collaborator

benoitf commented Jan 28, 2022

/retest

@benoitf
Copy link
Collaborator

benoitf commented Jan 28, 2022

it seems openshift ci is failing due to

[1/5] Validating package.json...
error chectl@0.0.2: The engine "node" is incompatible with this module. Expected version ">=16.0.0". Got "12.22.9"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

@benoitf

This comment has been minimized.

@tolusha
Copy link
Collaborator Author

tolusha commented Jan 28, 2022

I've changed it
https://github.com/che-incubator/chectl/pull/1986/files#diff-6704820b78e3aea8581dff31b66a527b85fc1e2635264f0c6961acb16529ff24
It must be another reason

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@tolusha
Copy link
Collaborator Author

tolusha commented Jan 28, 2022

/retest

@tolusha
Copy link
Collaborator Author

tolusha commented Jan 28, 2022

Fixed by updating NodeJs version in the main branch
#1988

@tolusha tolusha merged commit 0a7174a into main Jan 28, 2022
@tolusha tolusha deleted the 21075 branch January 28, 2022 10:52
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