asdqwex/discobot

View on GitHub
src/modules/lol.js

Summary

Maintainability
A
45 mins
Test Coverage

Function onMessage has 6 arguments (exceeds 4 allowed). Consider refactoring.
Open

  onMessage: function (bot, user, userID, channelID, message, rawEvent) {
Severity: Minor
Found in src/modules/lol.js - About 45 mins to fix

    Too many blank lines at the end of file. Max of 0 allowed.
    Open

    Severity: Minor
    Found in src/modules/lol.js by eslint

    disallow multiple empty lines (no-multiple-empty-lines)

    Some developers prefer to have multiple blank lines removed, while others feel that it helps improve readability. Whitespace is useful for separating logical sections of code, but excess whitespace takes up more of the screen.

    Rule Details

    This rule aims to reduce the scrolling required when reading through your code. It will warn when the maximum amount of empty lines has been exceeded.

    Options

    This rule has an object option:

    • "max" (default: 2) enforces a maximum number of consecutive empty lines.
    • "maxEOF" enforces a maximum number of consecutive empty lines at the end of files.
    • "maxBOF" enforces a maximum number of consecutive empty lines at the beginning of files.

    max

    Examples of incorrect code for this rule with the default { "max": 2 } option:

    /*eslint no-multiple-empty-lines: "error"*/
    
    var foo = 5;
    
    
    
    var bar = 3;

    Examples of correct code for this rule with the default { "max": 2 } option:

    /*eslint no-multiple-empty-lines: "error"*/
    
    var foo = 5;
    
    
    var bar = 3;

    maxEOF

    Examples of incorrect code for this rule with the { max: 2, maxEOF: 1 } options:

    /*eslint no-multiple-empty-lines: ["error", { "max": 2, "maxEOF": 1 }]*/
    
    var foo = 5;
    
    
    var bar = 3;

    Examples of correct code for this rule with the { max: 2, maxEOF: 1 } options:

    /*eslint no-multiple-empty-lines: ["error", { "max": 2, "maxEOF": 1 }]*/
    
    var foo = 5;
    
    
    var bar = 3;

    maxBOF

    Examples of incorrect code for this rule with the { max: 2, maxBOF: 1 } options:

    /*eslint no-multiple-empty-lines: ["error", { "max": 2, "maxBOF": 1 }]*/
    
    
    var foo = 5;
    
    
    var bar = 3;

    Examples of correct code for this rule with the { max: 2, maxBOF: 1 } options:

    /*eslint no-multiple-empty-lines: ["error", { "max": 2, "maxBOF": 1}]*/
    
    var foo = 5;
    
    
    var bar = 3;

    When Not To Use It

    If you do not care about extra blank lines, turn this off. Source: http://eslint.org/docs/rules/

    There are no issues that match your filters.

    Category
    Status