Skip to content

chore(ci): Remove all node18 usage #11252

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

usmanmani1122
Copy link
Contributor

closes: #8900
refs: #10979

Description

This PR removes all the Node 18 CI job runs and ensures all the CI jobs are running on Node 20

Security Considerations

None

Scaling Considerations

None

Documentation Considerations

None

Testing Considerations

CI should pass

Upgrade Considerations

None

@usmanmani1122 usmanmani1122 added the force:integration Force integration tests to run on PR label Apr 11, 2025
@usmanmani1122 usmanmani1122 requested review from turadg and mhofman April 11, 2025 08:28
@usmanmani1122 usmanmani1122 self-assigned this Apr 11, 2025
Copy link
Member

@mhofman mhofman left a comment

Choose a reason for hiding this comment

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

I'm not sure about the issue details, but we need to keep 2 variants in the matrix jobs, and just bump support to Node 20 & 22, which from what I understand has some blockers.

@usmanmani1122
Copy link
Contributor Author

usmanmani1122 commented Apr 11, 2025

I'm not sure about the issue details, but we need to keep 2 variants in the matrix jobs, and just bump support to Node 20 & 22, which from what I understand has some blockers.

Keeping this PR in draft until Node 22 is unblocked

@turadg turadg removed their request for review April 15, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

integration tests with Node 20
2 participants