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

v1.0.0 #140

Closed
wants to merge 1 commit into from
Closed

v1.0.0 #140

wants to merge 1 commit into from

Conversation

jacobp100
Copy link

I think this API is stable. Would it make sense to ship v1 and follow semver?

I think this API is stable. Would it make sense to ship v1 and follow semver?
@jacobp100
Copy link
Author

The reason for this is that one of the versions breaks on node 11.

aws-xray-sdk depends on this through aws-xray-sdk -> aws-xray-sdk-core -> node-continuation-local-storage -> this package.

It looks like node-continuation-local-storage is no longer developed, so it hasn't merged anything to bump the version of this package.

As a result, aws is switching out the node-continuation-local-storage package, even though that isn't the broken package.

If semver had been used, this would have automatically been fixed. We can't fix the issue with aws with this, but we can try and avoid this in the future! 😃

@rochdev
Copy link

rochdev commented Nov 19, 2018

Before releasing v1.0.0 it would probably be wise to fix the known memory leak when throwing from a native Promise handler (which is very common).

@jacobp100
Copy link
Author

1.0.0 just means API stability, it doesn't necessarily mean it's bug free! 😀

@carlisliu
Copy link

agreed, higher version of node.js should use native async_hooks module.

@rochdev
Copy link

rochdev commented Nov 20, 2018

@carlisliu async_hooks is only available in Node >=8 but native Promise was introduced in Node 4. This library is one of the only options from >=4 to <8 but is currently unusable because of the leak.

@Qard
Copy link
Collaborator

Qard commented Dec 3, 2018

v4 is EOL, and v6 is not far off from that too. I don't think motivation is very high to deal with super old versions of Node.js...

That being said, PRs are welcome. 🤷‍♂️

@jacobp100 jacobp100 closed this Nov 13, 2019
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.

4 participants