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

Change to Pydantic Version to v2 #20

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

dasTholo
Copy link

change ArangodbClient to python-arango
change python to >py3.10

dump pre-commit version
rename Graph to GrapModel (Name Collision)

dump Shylock Version
add Shylock AsyncDB Class
WIP EdgeModel Funcs like
UPSERT, get_many

change tests to python-arango
fixme
ENV Variable in pre-commit tests did not run

change ArangodbClient to python-arango
deprecate <py3.10

dump pre-commit version
rename Graph to GrapModel (Name Collision)

dump Shylock Version
add Shylock AsyncDB Class
WIP EdgeModel Funcs like
UPSERT, get_many

change tests to python-arango
fixme ENV Vars in pre-commit tests did not run
@dasTholo
Copy link
Author

I would be happy to receive comments.
This is my first project with Asyncer, so I'm not sure if it will make an improvement.
But aioarangodb is not maintained for a long time and python-arango regularly. Hence my decision to do it this way.

Looking forward to the exchange of thoughts!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this naming isn´t good..
Any suggestions?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

async_arangodb_backend.py is fine

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 this pull request may close these issues.

2 participants