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

Should we still recommend spaad in the README? #200

Closed
thomaseizinger opened this issue Jan 11, 2023 · 3 comments · Fixed by #212
Closed

Should we still recommend spaad in the README? #200

thomaseizinger opened this issue Jan 11, 2023 · 3 comments · Fixed by #212

Comments

@thomaseizinger
Copy link
Collaborator

At the moment, the README recommends to try spaad in case one wants better ergonomics out of xtra.

Personally, I was never a big fan of spaad because I've found that it obfuscates too many things. With the upcoming release, the public API of xtra changed massively and I believe it is now in a state where it provides great control over what is happening without too much boilerplate. For example, the split_receiver functionality is only available on SendFuture which would be hidden by something like spaad.

Especially once we add #111 and async fn in traits being hopefully stable soon, the amount of boilerplate is practically zero, i.e. every line of code is intentional and not a "this library is making me write code that I don't want to write".

@Restioson
Copy link
Owner

I am not 100% sure if we should recommend it per se but it might be worth a mention. I also agree. Spaad was always a little bit of an experiment with it and I was not totally satisfied with how it turned out.

@thomaseizinger
Copy link
Collaborator Author

Yeah, we should definitely mention it. What irks me is that the current wording suggests that xtra is too verbose to use by itself when at least in my opinion, it definitely isn't.

@Restioson
Copy link
Owner

I agree :) I think that macros are a little unergonomic and we should strive to improve xtra's usability outside of other macros like what was done in spaad.

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 a pull request may close this issue.

2 participants