Sign upLogin

marcells/node-build-monitor

View on GitHub
Star
app/public/scripts/AppViewModel.js

Summary

Maintainability
B
5 hrs
Test Coverage

Function AppViewModel has 101 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    var AppViewModel = function() {
        var self = this;

        var isLoadingInitially = true;
Severity: Major
Found in app/public/scripts/AppViewModel.js - About 4 hrs to fix

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

            this.updateCurrentBuildsWithChanges = function (changes)  {
            if (anyBuildMatchesToNotifcationFilter(changes.removed) ||
                anyBuildMatchesToNotifcationFilter(changes.added) ||
                anyBuildMatchesToNotifcationFilter(changes.updated)) {
                self.setHasUnreadBuilds(true);
    Severity: Minor
    Found in app/public/scripts/AppViewModel.js - About 1 hr to fix

      There are no issues that match your filters.

      Category
      Status