-
Notifications
You must be signed in to change notification settings - Fork 2
WIP: Doc/update #7
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
Conversation
Thanks. I might not have time to go through PRs recently. If everything is working locally and documents are generated correctly, then it should be fine 👍
@adlerhsieh Sorry, I have looked into retrieveMethodsFromNode, but it looks too complicated for me 😢
It might take more than half a month for the fix.
I'd be very glad if you'd fix the parser.
Yeap, I'll check it when I got time. Thanks for reporting!
Update commit id Update for loops in retrieveMethodsFromNode() Fix variable names Extract FnName and FnLine Parse comments, params, returns Remove redundant for loop Remove redundant for loop Remove redundant for loop
Finally updated retrieveMethodsFromNode().
Perhaps this should be working, but look like some api doc in Goby have been a bit inconsistent and needs fix.
Just to notify it is under construction of updating parser