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

Add --xloop-all feature #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tylerweitzman
Copy link

@tylerweitzman tylerweitzman commented Aug 14, 2024

@breuleux This new feature allows you to call --xloop-all as a CLI flag which will wrap all functions in the global namespace with xloop so that if any function in the script throws an error it gives you the opportunity to fix the error and keep going.

@tylerweitzman tylerweitzman changed the title Build xloop-all feature Add --xloop-all feature Aug 15, 2024
@breuleux
Copy link
Owner

Hey Tyler, sorry it took me a while to get to this. I tested it, this works well and seems useful to leverage the loop more easily. Just needs a ruff format, also a few # pragma: no cover to shut up the coverage check (this functionality is generally not covered right now since I'm not sure what the best way to test it would be). Thanks!

@tylerweitzman
Copy link
Author

I'll make the fixed.

Been using this a bunch and works well except when my debugger is attached. Readme calls out that limitation-- any ideas how we could conceivably overcome that limitation? Perhaps there's some way for jurigged to regain reloading power following a breakpoint by detecting resume?

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