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

v08 - fix ContinueAs data field assignment #92

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

spolti
Copy link
Member

@spolti spolti commented Oct 3, 2022

Signed-off-by: spolti filippespolti@gmail.com

Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:

Special notes for reviewers:

Additional information (if needed):

@ricardozanini ricardozanini removed the request for review from tsurdilo October 3, 2022 20:51
@spolti spolti force-pushed the continueAsValidation branch 2 times, most recently from f15f602 to fb52d75 Compare October 3, 2022 22:39
model/workflow.go Outdated Show resolved Hide resolved
@spolti spolti mentioned this pull request Oct 4, 2022
@spolti
Copy link
Member Author

spolti commented Oct 4, 2022

relates to #70

@spolti spolti force-pushed the continueAsValidation branch 2 times, most recently from cf35772 to c92800a Compare October 4, 2022 15:07
@spolti spolti requested review from lsytj0413 and removed request for ricardozanini October 4, 2022 15:08
@spolti
Copy link
Member Author

spolti commented Oct 4, 2022

interesting, I requested @lsytj0413 to re-review and it removed you @ricardozanini lol

@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@cc7f77a). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #92   +/-   ##
=======================================
  Coverage        ?   19.54%           
=======================================
  Files           ?       15           
  Lines           ?      655           
  Branches        ?        0           
=======================================
  Hits            ?      128           
  Misses          ?      492           
  Partials        ?       35           
Flag Coverage Δ
sdk-go 19.54% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

model/workflow.go Outdated Show resolved Hide resolved
model/workflow.go Show resolved Hide resolved
model/workflow.go Outdated Show resolved Hide resolved
model/workflow_ref.go Outdated Show resolved Hide resolved
@lsytj0413
Copy link
Collaborator

/lgtm

@spolti spolti force-pushed the continueAsValidation branch 2 times, most recently from 7ba8ffa to ac2546d Compare October 5, 2022 13:02
model/workflow.go Outdated Show resolved Hide resolved
Signed-off-by: spolti <filippespolti@gmail.com>
@ricardozanini ricardozanini merged commit 28e2e7d into serverlessworkflow:main Oct 5, 2022
@spolti spolti deleted the continueAsValidation branch October 5, 2022 13:26
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.

None yet

4 participants