Skip to content
This repository has been archived by the owner on Feb 19, 2024. It is now read-only.

Upgrade Node to v14 #77

Merged
merged 2 commits into from
Sep 28, 2020
Merged

Upgrade Node to v14 #77

merged 2 commits into from
Sep 28, 2020

Conversation

andygout
Copy link
Contributor

Please run appropriate tests and checks to ensure that the app is compatible with this Node version. https://node.green is a useful reference.

This PR was created using a nori script 🍙

Nori is a command-line application for managing changes across multiple (usually Github) repositories.

@andygout andygout requested a review from a team as a code owner September 23, 2020 07:31
Changes:

- `package.json` field `engines.node` - updated, previously: `>=8.11.0`
- `.nvmrc` - created
- `.circleci/config.yml` Docker image for `container_config_node` - Docker updated, previously: `circleci/node:8-browsers`
@andygout
Copy link
Contributor Author

Checks performed:

  • 🤷

No unit tests. 🤦

Ultimately this is not an app running in production rather than a series of scripts. If one of the scripts fails because of an incompatibility issue then it will simply mean that the script is not run, rather than a process in product failing.

@nickramsbottom mentioned this issue - Array.sort arguments order changed in Node.js 11 - so I have specifically checked that this repo's use of .sort still returns the latest index as it required by the function that calls it.

@andygout andygout merged commit c59cf90 into master Sep 28, 2020
@andygout andygout deleted the upgrade-node-to-v14 branch September 28, 2020 07:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants