-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Console] Remove deprecated console.exception event #22777
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
Conversation
mbabker
commented
May 19, 2017
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | no |
BC breaks? | yes |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | |
License | MIT |
Doc PR |
missing CHANGELOG updates in components (please check any your other PRs also if any) |
Added |
4.0.0 | ||
----- | ||
|
||
* [BC BREAK] removed `ConsoleExceptionEvent` |
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.
no need for these, we did add them in other PRs
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.
(I mean the BC BREAK prefix)
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.
Whatever PR I copied from had it, so that's what I rolled with. Rebased and removed the prefix.
Thank you @mbabker. |
…babker) This PR was merged into the 4.0-dev branch. Discussion ---------- [Console] Remove deprecated console.exception event | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Commits ------- 80f329c Remove deprecated console.exception event