-
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
Check variable existence in fetch #1151
Conversation
Jenkins go! |
Unit testing passed. |
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.
Do you add one UT case to cover this?
The result is not wrong, but an unreadable hint was given. |
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 taking care of this, please resolve the conflicts.
6e43c26
to
fa27612
Compare
Unit testing passed. |
Unit testing passed. |
Unit testing passed. |
1 similar comment
Unit testing passed. |
Unit testing passed. |
* fix wrong image path for architecture the original image path "https://github.com/vesoft-inc/nebula-docs/raw/master/images/Nebula%20Arch.png" is outdated and cannot be loaded by browsers. I used the image path for architecture from English README as a replacement. * change the architecture image path with given url vesoft-inc#4471 (comment) Co-authored-by: Sophie <[email protected]> Co-authored-by: TommyLemon <[email protected]> Co-authored-by: Sophie <[email protected]>
Close #1132, close #1135