Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Add support for node@12 #11

Merged
merged 2 commits into from
Nov 29, 2019
Merged

Add support for node@12 #11

merged 2 commits into from
Nov 29, 2019

Conversation

yvele
Copy link

@yvele yvele commented Sep 17, 2019

Requirements for Contributing a Bug Fix

Identify the Bug

Fixes #10

Description of the Change

This pull request adds supports for Node.js v12 by briging those 2 commits:

Alternate Designs

Not needed

Possible Drawbacks

I'm not into C++ but as @rsese said, it should be easy to implement 🤔 (edit by @rsese: as mentioned in #11 (comment), wasn't trying to say this update was easy 😄)

Verification Process

Added Node.js V12 in the unit test matrix

Release Notes

Add support for Node.js v12

@rsese

This comment has been minimized.

@rsese
Copy link

rsese commented Sep 20, 2019

Thanks for the contribution @yvele! Before it can be reviewed, can you please update the pull request body to use our pull request template? I think it may not have shown for you because this is a fork.

@yvele
Copy link
Author

yvele commented Sep 21, 2019

@rsese ok done

@rsese
Copy link

rsese commented Sep 30, 2019

Thanks @yvele - we can't promise a timeline for when this will get reviewed but if there are any questions about your changes, whoever reviews it will follow up.

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.

Build crashes with node 12
3 participants