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

Add node 20 to test matrix #2133

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

BackEndTea
Copy link

No description provided.

@netlify
Copy link

netlify bot commented Sep 20, 2023

Deploy Preview for nodemon ready!

Name Link
🔨 Latest commit 32807b5
🔍 Latest deploy log https://app.netlify.com/sites/nodemon/deploys/650aa8741d2b6b00081e6916
😎 Deploy Preview https://deploy-preview-2133--nodemon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@Abuchtela Abuchtela left a comment

Choose a reason for hiding this comment

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

commit 32807b5

Abuchtela

This comment was marked as duplicate.

Abuchtela

This comment was marked as duplicate.

@remy
Copy link
Owner

remy commented Oct 6, 2023

I don't think the tests are failing because of node 20 - but because something's either wrong in the test or there's a regression somewhere…

I'm trying to have a look but don't have a lot of time (on the computer) atm - but if no one else does, I will do and I will get node 20 in the matrix.

@remy remy added the not-stale Tell stalebot to ignore this issue label Oct 6, 2023
@jimmywarting
Copy link

How would you feel about just testing only the latest 3 versions? 16, 18, 20? and setting minium node version to it as well in package.json
Could start using newer NodeJS features by doing so.

ravin00

This comment was marked as outdated.

@remy
Copy link
Owner

remy commented Oct 26, 2024

Remove the node 10.x and add the 12.x support and upgrade the suppoted version to 18.x and 20.x. first of all do the checking first before the changes.

Why are you arbitrarily proposing to drop support for 10 and 12? (This is rhetorical actually and rather it's naive to assume that I'd want that)

@remy
Copy link
Owner

remy commented Oct 26, 2024

On reflection, it would be ideal to only run integration tests on the lowest common node version, rather than the latest with all the bells.

@ravin00
Copy link

ravin00 commented Oct 26, 2024

On reflection, it would be ideal to only run integration tests on the lowest common node version, rather than the latest with all the bells.

If that's the case your correct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-stale Tell stalebot to ignore this issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants