Enable PSR-2 check in /includes #4220

Merged
merged 1 commit into from Aug 28, 2016

Projects

None yet

4 participants

@murrant
Contributor
murrant commented Aug 26, 2016

Please note

Please read this information carefully.

The only one that could have a change in behaviour is hrstorage.inc.php ones... They had some oddly defined case statements (and duplicates)

@murrant murrant Enable PSR-2 check in includes
The only one that could have a change in behaviour is hrstorage.inc.php ones...  They had some oddly defined case statements (and duplicates)
ab734b6
@scrutinizer-notifier

The inspection completed: 22 new issues, 5 updated code elements

@laf laf added the Core label Aug 28, 2016
@laf laf merged commit 9284bc6 into librenms:master Aug 28, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@murrant murrant deleted the murrant:includes-psr2 branch Aug 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment