-
-
Notifications
You must be signed in to change notification settings - Fork 374
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
ScalaPB Plugin: Add option scala3_sources #3258
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable. Thank you! Do you intend to add a test case for that options? It's not a requirement, but helps with maintenance.
I've added a simple test, but I don't know if this is enough. If not, can you please give me a hint on how to improve it? |
@@ -182,6 +188,11 @@ object TutorialTests extends TestSuite { | |||
} | |||
} | |||
|
|||
"calledWithScala3Sources" - workspaceTest(TutorialWithScala3Soures) { eval => | |||
val result = eval.apply(Tutorial.core.compileScalaPB) | |||
assert(result.isRight) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@b-eyselein Is there something unique to the scala3_sources
option in the generated file(s)? you could assert some specific file content.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It mainly uses ?
instead of _
for unknown types which generates a bunch of compiler warnings...
Like the title says, this option allows users of the
scalapb
-Plugin to submit the optionscala3_sources
toscalapb
directly.