Comment parsing

Open ChristianToepfer opened this issue 6 years ago • 3 comments

Add comment parsing #136 for message and enum definition. I only use the doxygen comments ( /**.../,///) and skip normal developer comments (/...*/,//) to filter IDE-Help. IDE-Help with docstring for python classes and getter function. To be able to show any help for enum definition in python, I had to define the enums as a class. Help to single Enum-Item is still open.

Jun 06 '19 11:06 ChristianToepfer

This is great! Going to request some changes, but overall I think this looks good.

Jun 10 '19 00:06 tburmeister

This branch is failing tests/test_imported_enums.py, tests/test_items.py, tests/test_merge_from.py, and tests/test_parser/test_oneof.py for me. Should probably add a unit test for proper comment parsing as well.

Jun 15 '19 21:06 tburmeister

Tests are simply something fine. Two of them shown me a bug. I had to adapt a few others for class enum. Comment parsing tests coming soon.

Jul 09 '19 15:07 ChristianToepfer

remove comment parsing/providing with pyrobuf. Worked with pyi-files (pyrogen-stubs) much better

Nov 30 '22 15:11 ChristianToepfer