-
-
Notifications
You must be signed in to change notification settings - Fork 810
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
Decouple exceptions from exception handling #6361
Conversation
* @param Exception|null $previous | ||
* @param int $code | ||
*/ | ||
public function __construct($driver, $message = "", Exception $previous = null, $code = 0) |
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.
My OCD would like to know if that is copy/pasta? :wink
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.
Haha nope
6565fe0
to
3bcf272
Compare
… -4 (up from -8).
… upped the priority to -7
…esn't subscribe it if its disabled.
3bcf272
to
7c7a57c
Compare
CC builds are having issues with our ruleset … ignoring. Thanks for picking up the last of this chunk of work @CarsonF |
Probably a couple things left to finish, but here's the idea.