-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add documentation to proto files #34
Comments
Would it be a good idea to trace back what function mkwrapper is calling and use LinuxCNCs Python documentation as a base for the descriptions of the protobuf messages? That should cover the status, command and error messages. |
See https://github.com/machinekit/machinetalk-protobuf/pull/35/files#diff-07c5086a92949c3ffeb6a523cc39aa5dL119 |
@zultron Can you please add https://github.com/estan/protoc-gen-doc to the Machinekit repository. Seems to have proper deb package description, but is not in the repositories. |
More progress #51 |
@Strahlex is Qt5 really a requirement for building the docs? I would have to add this to the docs formatter container for automated builds :-/ https://github.com/mhaberler/docker-jekyll-asciidoctor pity it cannot do asciidoc |
tried this on ubuntu wiley:
any suggestions? edit: figured. Need to add qt5-default libprotoc-dev . Builds. |
Not sure whats the problem with the build but there are Ubuntu packages for this tool: https://software.opensuse.org/download.html?project=home%3Aestan%3Aprotoc-gen-doc&package=protoc-gen-doc sudo sh -c "echo 'deb http://download.opensuse.org/repositories/home:/estan:/protoc-gen-doc/xUbuntu_15.10/ /' >> /etc/apt/sources.list.d/protoc-gen-doc.list"
sudo apt-get update
sudo apt-get install protoc-gen-doc |
Should be added to README. |
Since this our Machinetalk API we should document the protbuf files and generate a documentation our of the source files e.g. with: https://github.com/estan/protoc-gen-doc
The text was updated successfully, but these errors were encountered: