diff options
author | Dylan William Hardison <dylan@hardison.net> | 2018-09-14 03:23:30 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-14 03:23:30 +0200 |
commit | 54ed8cf5c8a97f9aeccbac30870acebb0059a964 (patch) | |
tree | 401d9939b2d7272f83f8ed40b44df0810969e5fa /.circleci/config.yml | |
parent | d3e00529333fb2eadac9c22a45f20c1050401952 (diff) | |
download | bugzilla-54ed8cf5c8a97f9aeccbac30870acebb0059a964.tar.gz bugzilla-54ed8cf5c8a97f9aeccbac30870acebb0059a964.tar.xz |
no bug - cleanup a few nits in the SecurityRiskReport (#746)
- sorted imports, with Moo and MooX::StrictConstructor at the top because they
change the behavior of the code.
- removed 'scalar' when comparing an array to an integer as it isn't required.
- adjusted multi-line first { } to single line since it still fits and perltidy
makes it look ugly.
- store each 'result' hash in a $result variable, again to make perltidy format
better.
- change use of 'unshift ARRAY' to 'push ARRAY' and reverse().
The later performs fewer mallocs (push is much more effficient than unshift).
Please check if this logic is right.
Diffstat (limited to '.circleci/config.yml')
0 files changed, 0 insertions, 0 deletions