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

Add support for node protocol require/import. #11

Merged
merged 1 commit into from
May 11, 2021

Conversation

make-github-pseudonymous-again
Copy link
Contributor

Not sure this is the right place to implement this feature. I need something like this to continue working since xo v0.40.0 was released. See for instance nodejs/node#38343.

@make-github-pseudonymous-again
Copy link
Contributor Author

PS: A similar addition would be needed for the power-assert babel plugins and presets but I do not know where to start.

@twada
Copy link
Member

twada commented May 11, 2021

@make-github-pseudonymous-again Oh I didn't noticed this significant change. Thanks!!
I'll work on power-assert side.

@twada
Copy link
Member

twada commented May 11, 2021

LGTM 👍

@twada twada merged commit f381e8c into unassert-js:master May 11, 2021
@twada
Copy link
Member

twada commented May 11, 2021

@make-github-pseudonymous-again v3.1.0 is out. Thank you for your contribution!

Release v3.1.0: node: protocol (2021-05-11)

@make-github-pseudonymous-again
Copy link
Contributor Author

@twada Thanks a lot for your rapid action!

@make-github-pseudonymous-again
Copy link
Contributor Author

@twada Have you worked on power-assert? I can work on it if you want. I would need some pointers though.

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.

2 participants