-
Notifications
You must be signed in to change notification settings - Fork 127
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
fix(#3294): added PhiMojo
fails on error flag
#3295
fix(#3294): added PhiMojo
fails on error flag
#3295
Conversation
@yegor256 please check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rultor merge
@yegor256 Could you try to merge it one more time, please? It's quite urgent and blocks several issues. |
@rultor status |
@maxonfjvipon This is what's going on here:
More information about Rultor commands you can get here. |
@maxonfjvipon merged |
@maxonfjvipon Thanks for the contribution! You've earned +13 points for this: +30 as a basis; -7 for too many hits-of-code (235 >= 100); -10 for too few (4) comments. Please, keep them coming. Your running balance is +180. |
Ref: #3294
PR-Codex overview
The focus of this PR is to introduce a new parameter
phiFailOnError
inPhiMojo
to control error handling behavior.Detailed summary
phiFailOnError
parameter toPhiMojo
classPhiMojo
to handle errors based onphiFailOnError
value