-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Provide named exports, fixes #27 #37
Conversation
It's generally best to focus on the issue at hand and not cramp in lots of unrelated changes. I'm using XO and AVA in all my projects and I don't plan to remove them here. I also prefer keeping the default export to not break backwards compatibility, but also because I think it's a better API. |
node --test
for testing, fixes #27
@sindresorhus : Reverted all changes not essential to #27 . If you could take another look, that'd be great. If you're still not interested, go ahead and close I suppose. Cheers! |
You can use this trick to avoid having to specify the named exports manually for the default export: https://github.com/sindresorhus/query-string/blob/main/index.js |
Also for index.d.ts |
@sindresorhus Updated the *.d.ts file to use the same index/base pattern. Note that this uses the " |
Thanks :) |
[Deleted previous description.]
Adds support for named exports. This is achieved by declaring all functions and constants as named exports, first, and then using them to compose the default object API.
All that other guff about
ava
andxo
, etc... has been reverted. 😄Fixes #27