-
Notifications
You must be signed in to change notification settings - Fork 40
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
Guides 2 #357
Guides 2 #357
Conversation
under review |
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
Branch conflicts need to be resolved though before it can be merged.
@oiclid could you please resolve the conflicts |
@oiclid : are you still active on this PR? thanks |
@HoomanDgtl @andy108369 Sorry, I was not reading my notifications. Let me fix it. |
@HoomanDgtl @andy108369 Done |
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.
@HoomanDgtl looks good for merging.
No description provided.