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

Question: Is there a reason why pipeline is buried so far into the package? from expression.extra.result import pipeline #209

Open
ShravanSunder opened this issue Jul 12, 2024 · 2 comments

Comments

@ShravanSunder
Copy link

pipelines are currently in from expression.extra.result import pipeline

It seems like its a great way to chain results (or options) and i was wondering why its not exposed more easily. It seems to me like a core part of implementing result in the codebase.

@ShravanSunder ShravanSunder changed the title Quesiton: Is there a reason why pipeline is buried so far into the package? from expression.extra.result import pipeline Question: Is there a reason why pipeline is buried so far into the package? from expression.extra.result import pipeline Jul 12, 2024
@dbrattli
Copy link
Owner

Hi, it was added there since I was unsure if it should be added as it is in the current form (or not) i.e naming. But I guess we could now move it into the core module. The thing is that it's "container" specific so it needs to be scoped / namespaced e.g expression.result.pipeline since a Result pipeline is different from an Option pipeline.

@ShravanSunder
Copy link
Author

I guess pipeline is also missing async compatability

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

2 participants