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

OSlash dependency is not meant for production use #225

Closed
mikepii opened this issue Jan 12, 2022 · 5 comments · Fixed by #233
Closed

OSlash dependency is not meant for production use #225

mikepii opened this issue Jan 12, 2022 · 5 comments · Fixed by #233

Comments

@mikepii
Copy link
Contributor

mikepii commented Jan 12, 2022

jsonrpcserver uses the OSlash package. From the OSlash readme:

OSlash is intended to be a tutorial. For practical functional programming in Python in production environments you should use Expression instead.

@bcb
Copy link
Member

bcb commented Jan 13, 2022

Yes I’m keen to remove the dependency. One issue we found is that there’s no conda recipe for OSlash which in turn means no jsonrpcserver in conda.

We should use some other library or even write a simple bind function to include in jsonrpcserver. That’s the only part of OSlash being used.

@bcb
Copy link
Member

bcb commented Mar 28, 2022

I just found https://github.com/dry-python/returns which would be perfect for this.

bcb added a commit that referenced this issue May 14, 2022
@bcb
Copy link
Member

bcb commented May 24, 2022

This has been merged into version 6, which is coming soon.

@bcb bcb closed this as completed May 24, 2022
@antispam2002
Copy link

Any chances to get v6 out soon?

@bcb
Copy link
Member

bcb commented May 9, 2023

Yeah, all that’s left for 6 is to replace setup.py with pyproject.toml and some documentation and then I’ll make a pre-release.

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.

3 participants