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.

Sign up for GitHub

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

Jump to bottom

Require Non-capturing catch #51

Merged

simPod merged 1 commit into master from non-capt-catch

Dec 2, 2021

Contributor

simPod commented Nov 26, 2021

https://github.com/slevomat/coding-standard#slevomatcodingstandardexceptionsdisallownoncapturingcatch

try {
    changeImportantData();
} catch (PermissionException) { // The intention is clear: exception details are irrelevant
    echo "You don't have permission to do this";
}

grongor approved these changes

View reviewed changes


          Require Non-capturing catch

7a0afd9

simPod force-pushed the non-capt-catch branch from 9a253a3 to 7a0afd9 Compare

December 2, 2021 14:31

simPod merged commit 3e388b5 into master

simPod deleted the non-capt-catch branch

December 2, 2021 14:32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment