Sign upLogin

rooey/chromeos-filesystem-onedrive

View on GitHub
Star
src/scripts/window.js

Summary

Maintainability
A
2 hrs
Test Coverage

Function onClickedBtnMount has 33 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    onClickedBtnMount(evt) {
        const btnMount = document.querySelector('#btnMount');
        evt.preventDefault();
        btnMount.setAttribute('disabled', 'true');
        $.toast({
Severity: Minor
Found in src/scripts/window.js - About 1 hr to fix
  • Create a ticket

    Function assignEventHandlers has 27 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        assignEventHandlers() {
        const btnMount = document.querySelector('#btnMount');
        btnMount.addEventListener('click', e => {
            this.onClickedBtnMount(e);
        });
    Severity: Minor
    Found in src/scripts/window.js - About 1 hr to fix
    • Create a ticket

      Unnecessary semicolon.
      Open

      };
      Severity: Minor
      Found in src/scripts/window.js by eslint

      disallow unnecessary semicolons (no-extra-semi)

      Typing mistakes and misunderstandings about where semicolons are required can lead to semicolons that are unnecessary. While not technically an error, extra semicolons can cause confusion when reading code.

      Rule Details

      This rule disallows unnecessary semicolons.

      Examples of incorrect code for this rule:

      /*eslint no-extra-semi: "error"*/

var x = 5;;

function foo() {
    // code
};

      Examples of correct code for this rule:

      /*eslint no-extra-semi: "error"*/

var x = 5;

var foo = function() {
    // code
};

      When Not To Use It

      If you intentionally use extra semicolons then you can disable this rule.

      Related Rules

      There are no issues that match your filters.

      Category
      Status