Skip to content

Commit

Permalink
Merge pull request #228 from macbre/globalVariables-offenders
Browse files Browse the repository at this point in the history
globalVariables: offenders are a bit confusing
  • Loading branch information
macbre committed Feb 6, 2014
2 parents e415b6c + 06b4331 commit d0522af
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions modules/globalVariables/globalVariables.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@ exports.module = function(phantomas) {

phantomas.setMetric('globalVariables', globals.length);

if (globals.length > 0) {
phantomas.addOffender('globalVariables', globals.join(', '));
}
globals.forEach(function(varName) {
phantomas.addOffender('globalVariables', varName);
});
});
};

0 comments on commit d0522af

Please sign in to comment.