Skip to content

Conversation

charlievieth
Copy link

This eliminates the need to use the unsupported bou.ke/monkey package,
which fails to build on darwin/arm64. A better solution might be to remove
the use of os.Exit entirely since it's only called with os.Exit(0) and exiting
the program should be the responsibility of the caller to Complete.

@posener
Copy link
Owner

posener commented Feb 13, 2022

I think that this breaks the current package API. No?

This eliminates the need to use the unsupported bou.ke/monkey package,
which fails to build on darwin/arm64.
@charlievieth charlievieth force-pushed the cev/remove-monkey-patch branch from e127f63 to 53d82ec Compare July 17, 2025 22:05
@charlievieth
Copy link
Author

@posener This change does not break the API or any existing code since all it does is add an additional function to the API for controlling how the complete library causes the program to exit.

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