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

This is awesome! #1

Open
ladyleet opened this issue Mar 2, 2021 · 4 comments
Open

This is awesome! #1

ladyleet opened this issue Mar 2, 2021 · 4 comments

Comments

@ladyleet
Copy link

ladyleet commented Mar 2, 2021

@benlesh can we merge this into core?

@benlesh
Copy link

benlesh commented Mar 2, 2021

Well, since it's just a wrapper around tapwith a ternary statement, probably not. But I do like it. 🙂

@Coly010
Copy link
Owner

Coly010 commented Mar 2, 2021

I did consider doing it with operate, but then thought perhaps not.

What I can do, if you feel like there's a use case, is open a PR that does this correctly?

@benlesh

@benlesh
Copy link

benlesh commented Mar 3, 2021

The implementation is correct and fine. I don't think there's much to change. On the core library side, I think we're more likely to remove operators than add them.

@ladyleet
Copy link
Author

ladyleet commented Mar 3, 2021

@benlesh what ever happened to "community projects" area or something that we were talking about? do you think it's time to add "community projects" section to the docs!

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

No branches or pull requests

3 participants