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

Add support for Node 14. #289

Merged
merged 2 commits into from
Feb 4, 2022
Merged

Add support for Node 14. #289

merged 2 commits into from
Feb 4, 2022

Conversation

benbcai
Copy link
Contributor

@benbcai benbcai commented Feb 2, 2022

Summary

  • Add support for Node 14.
  • Upgrade to terra-functional-testing to V2.

Closes #140

Deployment Link

https://terra-applic-.herokuapp.com/

Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

@benbcai benbcai self-assigned this Feb 2, 2022
@benbcai benbcai temporarily deployed to terra-applic-node-14-bmmnlzclh February 2, 2022 22:58 Inactive
@benbcai benbcai temporarily deployed to terra-applic-node-14-bmmnlzclh February 3, 2022 06:29 Inactive
@@ -4,7 +4,7 @@
"version": "1.52.2",
"description": "A framework to support application development with Terra components",
"engines": {
"node": ">=8.9.2 <15"
"node": "^8.9.2 || ^10.13.0 || ^12.0.0 || ^14.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Four versions seems a bit much. We are supporting three versions in other places - why not make it easy on ourselves and drop 8.0. Node 8 is circa 2017.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We actually got rid of Node 8 when we upgraded this to Node 12. See change here. We later realized we broke our consumers because some are still on Node 8 so we added Node 8 back on this PR #227. So to continue to remain passive, we should continue to keep Node 8. We can remove it in our next MVB.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks. This motivates me to continue crafting a deprecation policy.

@xenoworf xenoworf self-requested a review February 4, 2022 18:02
@benbcai benbcai merged commit 9336977 into main Feb 4, 2022
@benbcai benbcai deleted the node-14 branch February 4, 2022 18:25
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.

Node 14 Upgrade
3 participants