Replies: 2 comments
-
CC @vlad107 |
Beta Was this translation helpful? Give feedback.
-
Thanks for detailing the use cases @itegulov! That was very helpful. I ran into the second use case when trying to enable explicit API mode for pbandk itself (for #134). I went ahead and added a I agree that a more flexible way via proto3 custom options could be useful for some situations as well, so that you can change the Kotlin visibility per-file, per-message/enum, or even per-field. But even if we supported configuring the visibility via custom options, I think we'd still want to keep the |
Beta Was this translation helpful? Give feedback.
-
Motivation
There are two main use cases that I can see for this feature:
internal
visibility modifier.public
visibility modifier.Implementation
We are particularly interested in the second use-case and have implemented a very straightforward way to add visibility modifiers to all generated classes: input-output-hk@5577e0a. But, as with #173, we might want to explore a more flexible way via custom proto3 options.
Beta Was this translation helpful? Give feedback.
All reactions