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: run tests on Node.js v20 #60

Merged
merged 2 commits into from
May 19, 2023
Merged

ci: run tests on Node.js v20 #60

merged 2 commits into from
May 19, 2023

Conversation

snitin315
Copy link
Contributor

What changes did you make? (Give an overview)

  • run tests on Node.js v20
  • Remove Node v19 and v17 from matrix

Is there anything you'd like reviewers to focus on?

I feel it's fine to test on even node version only.

@snitin315
Copy link
Contributor Author

Tests are failing, I will push a fix soon.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Apr 20, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: snitin315 / name: Nitin Kumar (85a8161)

@aladdin-add
Copy link
Member

the ci failing is related to node.js nodejs/node#47614.

@aladdin-add
Copy link
Member

esmock v2.2.3 has added node.js v20 support.

@aladdin-add aladdin-add merged commit 72ffdcd into main May 19, 2023
@aladdin-add aladdin-add deleted the snitin315-patch-1 branch May 19, 2023 03:24
@github-actions github-actions bot mentioned this pull request May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants