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

Support node v18 and drop node v12 #377

Merged
merged 2 commits into from May 1, 2022
Merged

Support node v18 and drop node v12 #377

merged 2 commits into from May 1, 2022

Conversation

devinivy
Copy link
Member

@devinivy devinivy commented May 1, 2022

  • Test on node v14+ and adopt some new syntax.
  • Update typescript and node v18-compatible versions of lab and code.

If this looks good, this will go out as hoek v10.

@devinivy devinivy added breaking changes Change that can breaking existing code dependency Update module dependency labels May 1, 2022
@devinivy devinivy added this to the 10.0.0 milestone May 1, 2022
@devinivy devinivy self-assigned this May 1, 2022
@devinivy devinivy merged commit 3a2cc0f into v10 May 1, 2022
@devinivy devinivy deleted the drop-node-12-support-18 branch May 1, 2022 17:55
@@ -1,5 +1,5 @@
Copyright (c) 2011-2020, Sideway Inc, and project contributors
Copyright (c) 2011-2014, Walmart
Copyright (c) 2011-2022, Sideway Inc, and project contributors
Copy link
Contributor

@hueniverse hueniverse May 1, 2022

Choose a reason for hiding this comment

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

Sideway copyright should be locked at 2011-2020 and project contributor moved to its own line like was done for Walmart.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good call, thanks for catching that. Fix is here: #378

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking changes Change that can breaking existing code dependency Update module dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants