Skip to content

feat(scalasdk): generate action sources for Scala SDK#447

Merged
patriknw merged 1 commit intomainfrom
rgc/actions-codegen-and-samples
Sep 22, 2021
Merged

feat(scalasdk): generate action sources for Scala SDK#447
patriknw merged 1 commit intomainfrom
rgc/actions-codegen-and-samples

Conversation

@octonato
Copy link
Copy Markdown
Member

No description provided.

@octonato octonato marked this pull request as ready for review September 21, 2021 20:02
Copy link
Copy Markdown
Member Author

@octonato octonato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated code still not compiling, but I believe this is a general issue in the codegen since the scala-value-entity-customer-registry sample is showing the same issues.

Comment on lines -76 to -80
private def isUnary(cmd: ModelBuilder.Command): Boolean = !cmd.streamedInput && !cmd.streamedOutput
private def isStreamIn(cmd: ModelBuilder.Command): Boolean = cmd.streamedInput && !cmd.streamedOutput
private def isStreamOut(cmd: ModelBuilder.Command): Boolean = !cmd.streamedInput && cmd.streamedOutput
private def isStreamInOut(cmd: ModelBuilder.Command): Boolean = cmd.streamedInput && cmd.streamedOutput
private def hasStream(cmd: ModelBuilder.Command): Boolean = isStreamIn(cmd) || isStreamOut(cmd) || isStreamInOut(cmd)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved into Command as we need it now in two different generators (Java and Scala)

@octonato octonato mentioned this pull request Sep 21, 2021
21 tasks
@octonato octonato force-pushed the rgc/actions-codegen-and-samples branch from a73643d to 0fcff33 Compare September 21, 2021 20:09
Copy link
Copy Markdown
Contributor

@patriknw patriknw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking good...

@octonato octonato force-pushed the rgc/actions-codegen-and-samples branch from 0fcff33 to 56b9ef2 Compare September 22, 2021 12:08
Copy link
Copy Markdown
Contributor

@patriknw patriknw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, there will be some conflicts with #441 but I will solve those

@patriknw patriknw merged commit 803eba7 into main Sep 22, 2021
@patriknw patriknw deleted the rgc/actions-codegen-and-samples branch September 22, 2021 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants