-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fixed issue #74 of keyword error #76
fixed issue #74 of keyword error #76
Conversation
Can one of the admins verify this patch? |
jenkins, please run tests |
Build succeeded. |
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. +1
Changes about graph.thrift. Maybe @dutor want to review it again.
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 fixing this bug 👍
The name bool_val would be better. Thanks for your review @sherman-the-tank @dangleptr |
jenkins, please run tests |
Build succeeded. |
jenkins go |
Build succeeded. |
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.
Thank you for the contribution!
* fixed issue vesoft-inc#74 of keyword error * change var name from bool_flag to bool_val
* fixed issue vesoft-inc#74 of keyword error * change var name from bool_flag to bool_val
Fix hyperlink error Co-authored-by: SuperYoko <90179377+SuperYoko@users.noreply.github.com>
issue #74 fixed.