-
Notifications
You must be signed in to change notification settings - Fork 13
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
Implemented support for creating neogma models with decorators #75
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spectacular work, looking forward to this being merged! 🚀
I've added a few comments, mostly related to code and types, as I don't have the time to test its functionality. However, since you've added tests, it works out great.
Some of my comments are in the form of questions and might not be relevant, but in this case please reply them so I'm aware of whatever I'm missing :)
Again, this is amazing!!
…d setter args Co-authored-by: Jason Athanasoglou <jathanasoglou@outlook.com>
Co-authored-by: Jason Athanasoglou <jathanasoglou@outlook.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes, I've added a few more comments to continue our conversation :)
Hi all! I am looking forward to this PR, are there any updates on this? |
Any updates? |
No description provided.