Skip to content

Conversation

landrok
Copy link
Contributor

@landrok landrok commented Sep 13, 2016

L116
} catch (Exception $e) { // can't be caught because CodeClimate\PHPMD\Exception does not exist
Another way to fix this is:
} catch (\Exception $e) {

L116 
```        } catch (Exception $e) { // can't be caught because CodeClimate\PHPMD\Exception does not exist```
Another way to fix this is:
```        } catch (\Exception $e) {```
@wfleming
Copy link
Contributor

wfleming commented Oct 6, 2016

Thanks for this and the other PRs you opened @landrok: sorry they got missed and haven't been responded to yet. These all look like good catches, so I'm going to be merging them shortly. Thanks for the fixes!

@wfleming wfleming merged commit c17b55c into codeclimate:master Oct 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants