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 a chain method #80

Open
vitormcruz opened this issue Jul 7, 2023 · 2 comments
Open

Add a chain method #80

vitormcruz opened this issue Jul 7, 2023 · 2 comments

Comments

@vitormcruz
Copy link

So that one can use:

(1 2 3 4 5 6 7 8 9) chain select: [ :num | num even ];
                                       collect: [ :a | a / 2].

instead of:

((#(1 2 3 4 5 6 7 8 9) select: [ :num | num even ])
collect: [ :a | a / 2]).

If interesting, I have an implementation that uses Avatar that I can provide. The downside is that they don't have a proper baseline, so I had to fork the project in order to add one, issued a pull request, but up until today they did not integrated it: guillep/avatar#14

@gcotelli
Copy link
Member

Looks like a nice addition. Given that this library is a dependency of all the others in the ba-st ecosystem we are extra careful with adding dependencies. We need to take into account that if we're going to depend on another library we want some minimal things from it:

  • A working CI setup that runs the library tests
  • Alignment with the supported Pharo versions
  • A sane release policy, so we can depend on versions or branches where breaking changes are clearly identified so our build doesn't stop working for external reasons

@gcotelli gcotelli changed the title Ad a chain method Add a chain method Jul 17, 2023
@vitormcruz
Copy link
Author

Problem is that the best implementation I came up so far was using avatar (it is fast, general and easy to understand). I don't see they working on it anytime soon since they didn't answered my requestb and I know some of them are busy with other projects.

Do I create a CI in my fork anyway with a release so to frozen something more or less stable? Another alternative is incorporate avatar into buoy, but I don't know if it is interesting or if the original author would be ok with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants