Skip to content

feat: support to subflows on the Java DSL#1213

Open
matheusandre1 wants to merge 1 commit intoserverlessworkflow:mainfrom
matheusandre1:issue-1151
Open

feat: support to subflows on the Java DSL#1213
matheusandre1 wants to merge 1 commit intoserverlessworkflow:mainfrom
matheusandre1:issue-1151

Conversation

@matheusandre1
Copy link

Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:
Closes: #1151

Special notes for reviewers:

Additional information (if needed):

@matheusandre1 matheusandre1 marked this pull request as ready for review March 7, 2026 01:01
@matheusandre1 matheusandre1 requested a review from fjtirado as a code owner March 7, 2026 01:01
Copilot AI review requested due to automatic review settings March 7, 2026 01:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copy link
Collaborator

@mcruzdev mcruzdev left a comment

Choose a reason for hiding this comment

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

It looks great @matheusandre1, could you add a test running a workflow with your changes?

Copilot AI review requested due to automatic review settings March 10, 2026 22:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Signed-off-by: Matheus André <92062874+matheusandre1@users.noreply.github.com>
Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Looks solid, we just need to add the Function to the input or it's useless for the FuncDSL (unless jq scripting).

@fjtirado
Copy link
Collaborator

Looks solid, we just need to add the Function to the input or it's useless for the FuncDSL (unless jq scripting).

@ricardozanini The Function should be located at the experimental module, I think it can be another PR.

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Indeed, the Function can be added to a new PR - or directly to this one. Not sure why not. I do the DSL changes in both sides all the time since one influence the other.

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.

Add support to subflows on the Java DSL

5 participants