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

chore: export Arguments #11

Merged
merged 1 commit into from
Aug 15, 2020
Merged

chore: export Arguments #11

merged 1 commit into from
Aug 15, 2020

Conversation

Alon-L
Copy link
Contributor

@Alon-L Alon-L commented Aug 14, 2020

I find the Arguments type useful for dynamic event calls that require the arguments to be passed onto the listener.
I believe they should be exported as they serve as a core and useful type in the library.

Copy link
Owner

@andywer andywer left a comment

Choose a reason for hiding this comment

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

Fine with me 👍

@andywer
Copy link
Owner

andywer commented Aug 15, 2020

Thanks for contributing, guys!

@andywer andywer merged commit 1785367 into andywer:master Aug 15, 2020
@andywer
Copy link
Owner

andywer commented Aug 15, 2020

Do you need the change released quickly?

@Alon-L
Copy link
Contributor Author

Alon-L commented Aug 15, 2020

Yes I do. Thank you for approving this PR!

@erfanium
Copy link
Contributor

erfanium commented Aug 15, 2020

@Alon-L You can install node packages directly from github with npm: https://medium.com/@jonchurch/use-github-branch-as-dependency-in-package-json-5eb609c81f1a

This is so useful to me, I no longer have to wait for the changes to be released

@andywer
Copy link
Owner

andywer commented Aug 16, 2020

Glad you found a quick fix. Released now as v1.3.0, too 😉

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.

3 participants