State does not store workflow or context

ManageIQ / floe

Floe is a runner for Amazon States Language workflows

Apache License 2.0

0 stars 5 forks source link

State does not store workflow or context #210

Closed kbrock closed 2 months ago

kbrock commented 2 months ago

depends upon:

[ ] #209
[x] #203

When you setup a Parallel state, a single State may be run with multiple Context values.

So this changes State to pass the Context on execute (so a single State can handle multiple flows. - a kind of command/flyweight pattern if you will.

This is split up into quite a few commits to try and make the progression more palatable.

kbrock commented 2 months ago

update:

depends upon common errors
no longer passing workflow into State.new/State.build - passing validator instead

kbrock commented 2 months ago

update:

rebase

miq-bot commented 2 months ago

This pull request is not mergeable. Please rebase and repush.

kbrock commented 2 months ago

update:

rebase (moved a bunch of this logic into parent #209 )
due to changes in parent, now also don't pass workflow into build (validator handles that case)

miq-bot commented 2 months ago

Checked commits https://github.com/kbrock/floe/compare/41d484825d3daa2587588777073ff01797cdf716~...cc48d3871f1ae0ad7964bfec8249296889f41414 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 25 files checked, 0 offenses detected Everything looks fine. :+1:

kbrock commented 2 months ago

merged #216 instead