Nadav Samet
Nadav Samet
I came across the same issue when adding Scala, and the workaround of the code snippet above (adding `java`) worked for me as well. I initially thought it's a [Prism...
@SandroGrzicic thanks for fixing this. We haven't heard anything for a while, and wanted to have a shot at addressing few other gaps (lenses, one-ofs, custom types), so we started...
``` ➜ dpkg -S /usr/lib/x86_64-linux-gnu/qt5/qml/QtQuick/Controls.2/Plasma/ plasma-framework: /usr/lib/x86_64-linux-gnu/qt5/qml/QtQuick/Controls.2/Plasma ➜ dpkg -l | grep plasma-framework ii plasma-framework 5.38.0-0ubuntu1 amd64 Plasma Runtime components```
I think it's going to be tricky - you need to somehow find the user's extension through `enumValue.getOptions()`. It would have been straightforward if the Java code was already generated...
One thing you can try is to work in `compiler-plugin/src/test/scala/scalapb/compiler/ProtoValidationSpec.scala` - the method runValidation there loads protos from strings (by running protoc), and gives back descriptors. I'm not sure if...
Thanks for posting. Can you add to the proposal the following: * how the duplicator will be selected? * Would having a deduper require the field to be lazy evaled...
Hi @markosski , thanks for reporting. For messages, we have an `annotations` option that allow users to add a `private`. I will be happy to review a PR that adds...
Sorry for the delay. I am not comfortable with this proposal since inferred type for users' enum vals is going to become `Recognized` rather than `EnumName`.
The concern is this: ``` val x = Seq(Weather.RAIN, Weather.SUNSHINE) ``` The inferred type of `x` would be `Seq[Weather.Recognized]`, or `Seq[WeatherRecognized]` (if we go with the proposal from the previous...
@mantasindrasius , thanks for reporting! I confirm that unknown fields are not converted to/from java for any ScalaPB message. Will you be able to work on a PR for this?