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

Allow bake fixture all to fail without exiting #86

Open
adayth opened this issue May 7, 2015 · 6 comments
Open

Allow bake fixture all to fail without exiting #86

adayth opened this issue May 7, 2015 · 6 comments

Comments

@adayth
Copy link

adayth commented May 7, 2015

If you try to generate all fixtures with bake fixture all -f and one generation fails, for example because table cannot be detected for a model or anything else, that fail should not abort the 'all' process.

Maybe this could be happening with all other bake **** all options.

@dereuromark
Copy link
Member

Indeed, maybe wrapping with try catch could help here.

@dereuromark
Copy link
Member

Relates a bit to #427 - as this will minimize the issue at least.

@dereuromark
Copy link
Member

I wrap in the same way for IdeHelper using try catch - this should also be done here for all to prevent this from being not user friendly.

@othercorey
Copy link
Member

@markstory Do we want this to not fail? I'm not sure users would recognize the failure happened if we ignore it and successfully complete. Baking all could generate a lot of output.

@markstory
Copy link
Member

I'm not sure users would recognize the failure happened if we ignore it and successfully complete. Baking all could generate a lot of output.

That is my concern as well. We would need to provide a summary at the end the run as errors could be missed.

@github-actions
Copy link

This issue is stale because it has been open for 120 days with no activity. Remove the stale label or comment or this will be closed in 15 days

@github-actions github-actions bot added the stale label Jan 13, 2023
@othercorey othercorey added pinned and removed stale labels Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants