Skip to content

Conversation

@kzantow
Copy link
Contributor

@kzantow kzantow commented Oct 16, 2025

Description

This PR updates the go-sync library to return panics as errors during concurrent calls, and adds an option to loudly log, but not fail on errors. This option will allow users to opt in to behavior in their CI environment, for example, that errors including panics will not cause the process to return nonzero, but instead only be error logged.

Type of change

  • Feature

Checklist:

  • I have added unit tests that cover changed behavior
  • I have tested my code in common scenarios and confirmed there are no regressions
  • I have added comments to my code, particularly in hard-to-understand sections

TODO:

  • configuration to continue in the event of a non-unknown error bubbling up to the top with --continue-on-error or similar

Signed-off-by: Keith Zantow <[email protected]>
Signed-off-by: Keith Zantow <[email protected]>
@kzantow kzantow marked this pull request as draft October 20, 2025 16:14
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.

2 participants