-
Notifications
You must be signed in to change notification settings - Fork 11
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
feat: create scylla-getting-started-golang #27
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.
Hey, @Canhassi12. Good job! I added some comments to help you to improve the structure and the code a little bit. Hope it helps.
LGTM!! |
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.
I've spent some time with you reviewing each item and comment I left here. In the end, we conducted a test to ensure that all system functionalities were cohesive.
There are a few minor points that could be improved, but they certainly don't prevent the contribution from being accepted - it can be improved in a separate PR since these points are just polishing. The approach is quite straightforward, and the separation of layers is coherent.
So, in summary... LGTM 😄
LGTM |
Congratulations!!! I learned a lot with this PR |
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.
LGTM! Thanks for the contribution.
Hi there!
This PR aims to add documentation and the base project for using ScyllaDB with Golang.
I hope it was practical! If any modifications are needed, please let me know!