Skip to content
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

relax state action validation #183

Merged
merged 1 commit into from Aug 24, 2023

Conversation

spolti
Copy link
Member

@spolti spolti commented Aug 24, 2023

The purpose of this change is to align the workflow validationb between the Java and the GO SDK. Spec allows the action to be a empty list while in the GO SDK, if defined, must have at least one item.

Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:

Special notes for reviewers:

Additional information (if needed):

The purpose of this change is to align the workflow validationb between
the Java and the GO SDK. Spec allows the action to be a empty list while
in the GO SDK, if defined, must have at least one item.

Signed-off-by: Spolti <filippespolti@gmail.com>
@ricardozanini ricardozanini merged commit 796270c into serverlessworkflow:main Aug 24, 2023
7 checks passed
@ricardozanini
Copy link
Member

Feel free to release a new patch version.

@spolti spolti deleted the relaxActionValidation branch August 25, 2023 13:57
@spolti
Copy link
Member Author

spolti commented Aug 25, 2023

@ricardozanini sure, maybe we cal fix this #179 before?

@ricardozanini
Copy link
Member

@ricardozanini sure, maybe we cal fix this #179 before?

Agreed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants