Skip to content

Conversation

@bruxodasilva
Copy link
Contributor

Summary

Reenforce that for NonRetryableErrors, users need to manage the error or the Workflow will fail

@github-actions github-actions bot added the product:workflows Workflows: https://developers.cloudflare.com/workflows/ label Oct 28, 2024
@bruxodasilva bruxodasilva requested a review from elithrar October 28, 2024 18:21
@github-actions github-actions bot added size/s and removed size/xs labels Oct 28, 2024
Copy link
Collaborator

@elithrar elithrar left a comment

Choose a reason for hiding this comment

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

.

@bruxodasilva bruxodasilva requested a review from elithrar October 30, 2024 18:16
@bruxodasilva
Copy link
Contributor Author

@cloudflare/pcx-technical-writing can you please check this and merge?

@pedrosousa
Copy link
Contributor

@bruxodasilva I've approved the PR, but I can't merge it since @elithrar requested changes.
He must approve or else we'll need a repo admin to merge.

@bruxodasilva bruxodasilva merged commit 0002cf9 into cloudflare:production Nov 14, 2024
7 checks passed
@bruxodasilva bruxodasilva deleted the dferreira/workflows/improve-nonretryableerror-section branch November 14, 2024 16:29
harshil1712 pushed a commit that referenced this pull request Dec 3, 2024
* Reenforce that for NonRetryableErrors, users need to manage the error or the Workflow will fail
* Added code example to catch NonRetryableError

---------

Co-authored-by: Sidhartha Chatterjee <[email protected]>
Co-authored-by: Matt Silverlock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workflows Workflows: https://developers.cloudflare.com/workflows/ size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants