-
Notifications
You must be signed in to change notification settings - Fork 68
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
Improvements, and plans for the future. #68
Comments
At the moment, we are working on SQLAlchemy 1.4/2.0 support. There are a couple of PRs open but they haven't been thoroughly tested. |
Awesome! I'll go through PR's tomorrow, and also plan to write my thoughts regarding improvements, etc. |
Ok, from my side I want to share the next things, to consider them as new features/improvements.
Also, this list will be updated, once new ideas will arrive. |
Thanks @myusko. I'll add to the list as soon as I'm home. |
To continue where @myusko left off,
|
@michaelbukachi I totally agree that mypy should be added, let's create an issue for it, also in the documentation, we also should mention that the minimum Python version is 3.5. Regarding, the conjunction operators, I also agree, and definitely, we should strive for simplicity, as you mentioned. |
@myusko I've updated the readme. |
@michaelbukachi you should specify that version needs to be 3.5 and above |
@SK-FComputer I'll update this, thank you! |
Are there plns to support async sqlalchemy i future? |
Yeah, same question |
@aurthurm @ZhymabekRoman async is quite stable on SQAlchemy and is still heavily under development. We'd rather wait till it's completely stable rather than do a lot of fire fighting with each SQLAlchemy release. |
@michaelbukachi we can have an async branch for async functionality and not use master branch so that we can start brainstorming the async functionality. Those interested in using the features can use as we wait for the stable version. WIll be happy to share how i refactored the mixins to suport async functinality. My approach might not have been the best but am willing to put it on the table and open for suggestions. |
Hey, @aurthurm sure thing. Looking forward to your PR! |
Hie @michaelbukachi, I have added an issue and a pull request to this. Awaiting your insights |
@michaelbukachi, we should collect all suggestions/thoughts from the community, on which things we should focus on in the future, which features we should implement, and what type of improvements the community would like to see sqlalchemy-mixins.
Any comments/suggestions from the community are welcomed!
The text was updated successfully, but these errors were encountered: