Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ci: add old node versions to CI#1113

Open
fedeci wants to merge 1 commit intomasterfrom
fedeci/add-old-node-versions-to-ci
Open

ci: add old node versions to CI#1113
fedeci wants to merge 1 commit intomasterfrom
fedeci/add-old-node-versions-to-ci

Conversation

@fedeci
Copy link
Member

@fedeci fedeci commented Dec 1, 2021

Description

Related to #1112

To-do list

  • I have added tests for what I changed.
  • This pull request is ready to merge.

@fedeci
Copy link
Member Author

fedeci commented Dec 1, 2021

Oh, the version of jest we are using is not supporting node 8, I will fix it.

Copy link
Member

@gustavohenke gustavohenke left a comment

Choose a reason for hiding this comment

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

I don't think we need to support such old versions. They have reached EOL already.

Also parts of the code may not work, e.g. https://github.com/express-validator/express-validator/releases/tag/v6.4.0 this might be gibberish

@fedeci
Copy link
Member Author

fedeci commented Dec 2, 2021

We are still mentioning support for Node 8 both in the readme and in the package.json. I 100% agree we should remove it (maybe also Node 10), but we need to wait for a major to definitely cancel them.

@gustavohenke
Copy link
Member

Ouch this has totally been an oversight of mine. I don't think that it'd work on Node 8 anyway.

@gustavohenke gustavohenke dismissed their stale review December 3, 2021 23:46

I totally misread the release link I posted before 🀦 it might actually work, but I still don't think we should support Node <12. WDYT we should do?

@gustavohenke gustavohenke force-pushed the master branch 2 times, most recently from 3177f64 to 6e160b1 Compare April 7, 2023 12:26
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.

2 participants