Skip to content

add description of workflow sample#961

Closed
MregXN wants to merge 2 commits intodapr:masterfrom
MregXN:add-description-to-workflow-sample
Closed

add description of workflow sample#961
MregXN wants to merge 2 commits intodapr:masterfrom
MregXN:add-description-to-workflow-sample

Conversation

@MregXN
Copy link
Member

@MregXN MregXN commented Nov 22, 2023

Description

Added some descriptions to help understand the workflow sample

Signed-off-by: MregXN <mregxn@gmail.com>
@MregXN MregXN requested review from a team as code owners November 22, 2023 15:14
@kaibocai
Copy link
Contributor

There is a PR to improve the whole examples of workflow #949. Please review and see if we can close this one. Thanks.

@codecov
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3b0747c) 76.54% compared to head (dab57e8) 76.54%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #961   +/-   ##
=========================================
  Coverage     76.54%   76.54%           
  Complexity     1467     1467           
=========================================
  Files           137      137           
  Lines          4472     4472           
  Branches        527      527           
=========================================
  Hits           3423     3423           
  Misses          763      763           
  Partials        286      286           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cicoyle
Copy link
Contributor

cicoyle commented Dec 21, 2023

@MregXN - did you see the updated workflows examples @kaibocai added? The examples look to have more details, but if you still feel like you can add to the go for it. In the mean time I think we can close this out. Feel free to reopen or open a new PR updating the new examples if you feel inclined to do so 👍🏻

@cicoyle cicoyle closed this Dec 21, 2023
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.

4 participants