sabledocs icon indicating copy to clipboard operation
sabledocs copied to clipboard

I want to generate docs for proto file which contains thousands of messages

Open awakair opened this issue 1 year ago • 1 comments

Hi! I want to generate docs for proto file which contains thousands of messages and display in index.html only small part of them. But I still want clickable links to packages. This PR will add such functional to your tool

awakair avatar Oct 17 '24 16:10 awakair

@awakair Thanks for the patience on this!

I checked the PR, but it's not clear to me what we're trying to achieve. In the description you're mentioning that you have thousands of messages, and you want to display only a subset of them. But in the change we're not limiting the messages we're displaying, but rather the packages, aren't we? (And in the README you use the key MyMessage, but then you search in the dictionary with package.name, right?) And do I understand correctly that it's also adding the capability to override the name of a package to a different name which will be displayed in the list? This seems like a strange requirement, what is the use case in which you'd want to override a package name?

(Besides, I'm not sure about this implementation, I don't like that we'd be using one config setting for two different purposes, both for controlling which packages are shown in the list, and also overriding the names, I'd rather separate those two capabilities.)

markvincze avatar Dec 07 '24 10:12 markvincze