Paths

Table of Contentst

Differential D314

Don't report CommandNotFoundException
ClosedPublic
Actions

Authored by dereckson on Mar 2 2016, 11:02.

Details

Reviewers

dereckson

Maniphest Tasks

T765: Don't log CommandNotFoundException

Commits

rNOTIF854cb807bddf: Don't report CommandNotFoundException

Summary

Fixes T765.

Test Plan

Run php artisan tinekr (misspelling for tinker)
Check it's not reported to Sentry

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Passed

Unit

Tests Passed

Branch

T765

Event Timeline

dereckson updated this revision to Diff 740.Mar 2 2016, 11:02

dereckson added a task: T765: Don't log CommandNotFoundException.

dereckson retitled this revision from to Don't report CommandNotFoundException.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

Build started: https://ci.nasqueron.org/job/test-notifications-phab/97/ (console: https://ci.nasqueron.org/job/test-notifications-phab/97/console)

Build is green https://ci.nasqueron.org/job/test-notifications-phab/97/ for more details.

dereckson accepted this revision.Mar 2 2016, 11:23

dereckson edited edge metadata.

This revision is now accepted and ready to land.Mar 2 2016, 11:23

Closed by commit rNOTIF854cb807bddf: Don't report CommandNotFoundException (authored by dereckson, committed by dereckson). · Explain WhyMar 2 2016, 11:23

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

app/

Exceptions/

Handler.php

2 lines

90%

Diff 740

View Options

app/Exceptions/Handler.php

Don't report CommandNotFoundExceptionClosedPublicActions