fomantic-ui 2.9.1-beta.23 → 2.9.1-beta.25

package/src/definitions/modules/tab.js

@@ -12,14 +12,14 @@
     'use strict';
 
     function isWindow(obj) {
-        return obj != null && obj === obj.window;
+        return obj !== null && obj === obj.window;
     }
 
     function isFunction(obj) {
         return typeof obj === 'function' && typeof obj.nodeType !== 'number';
     }
 
-    window = (window !== undefined && window.Math === Math)
+    window = window !== undefined && window.Math === Math
         ? window
         : globalThis;
 
@@ -35,7 +35,7 @@
             performance     = [],
 
             query           = arguments[0],
-            methodInvoked   = (typeof query === 'string'),
+            methodInvoked   = typeof query === 'string',
             queryArguments  = [].slice.call(arguments, 1),
 
             initializedHistory = false,
@@ -44,7 +44,7 @@
 
         $allModules.each(function () {
             var
-                settings        = ($.isPlainObject(parameters))
+                settings        = $.isPlainObject(parameters)
                     ? $.extend(true, {}, $.fn.tab.settings, parameters)
                     : $.extend({}, $.fn.tab.settings),
 
@@ -94,12 +94,12 @@
                     }
 
                     var activeTab = module.determine.activeTab();
-                    if (settings.autoTabActivation && instance === undefined && activeTab == null) {
+                    if (settings.autoTabActivation && instance === undefined && activeTab === null) {
                         activeTab = settings.autoTabActivation === true ? module.get.initialPath() : settings.autoTabActivation;
                         module.debug('No active tab detected, setting tab active', activeTab);
                         module.changeTab(activeTab);
                     }
-                    if (activeTab != null && settings.history) {
+                    if (activeTab !== null && settings.history) {
                         var autoUpdate = $.address.autoUpdate();
                         $.address.autoUpdate(false);
                         $.address.value(activeTab);
@@ -193,7 +193,7 @@
 
                         return false;
                     }
-                    if (settings.historyType == 'state') {
+                    if (settings.historyType === 'state') {
                         module.debug('Using HTML5 to manage state');
                         if (settings.path !== false) {
                             $.address
@@ -259,7 +259,7 @@
                 cache: {
 
                     read: function (cacheKey) {
-                        return (cacheKey !== undefined)
+                        return cacheKey !== undefined
                             ? cache[cacheKey]
                             : false;
                     },
@@ -286,7 +286,7 @@
                 set: {
                     auto: function () {
                         var
-                            url = (typeof settings.path === 'string')
+                            url = typeof settings.path === 'string'
                                 ? settings.path.replace(/\/$/, '') + '/{$tab}'
                                 : '/{$tab}'
                         ;
@@ -323,11 +323,11 @@
 
                 changeTab: function (tabPath) {
                     var
-                        pushStateAvailable = (window.history && window.history.pushState),
-                        shouldIgnoreLoad   = (pushStateAvailable && settings.ignoreFirstLoad && firstLoad),
-                        remoteContent      = (settings.auto || $.isPlainObject(settings.apiSettings)),
+                        pushStateAvailable = window.history && window.history.pushState,
+                        shouldIgnoreLoad   = pushStateAvailable && settings.ignoreFirstLoad && firstLoad,
+                        remoteContent      = settings.auto || $.isPlainObject(settings.apiSettings),
                         // only add default path if not remote content
-                        pathArray = (remoteContent && !shouldIgnoreLoad)
+                        pathArray = remoteContent && !shouldIgnoreLoad
                             ? module.utilities.pathToArray(tabPath)
                             : module.get.defaultPathArray(tabPath)
                     ;
@@ -338,7 +338,7 @@
                             currentPath        = module.utilities.arrayToPath(currentPathArray),
 
                             isTab              = module.is.tab(currentPath),
-                            isLastIndex        = (index + 1 == pathArray.length),
+                            isLastIndex        = index + 1 === pathArray.length,
 
                             $tab               = module.get.tabElement(currentPath),
                             $anchor,
@@ -358,7 +358,7 @@
                             } else {
                                 nextPathArray = pathArray.slice(0, index + 2);
                                 nextPath = module.utilities.arrayToPath(nextPathArray);
-                                isLastTab = (!module.is.tab(nextPath));
+                                isLastTab = !module.is.tab(nextPath);
                                 if (isLastTab) {
                                     module.verbose('Tab parameters found', nextPathArray);
                                 }
@@ -392,7 +392,7 @@
                                 settings.onFirstLoad.call($tab[0], currentPath, parameterArray, historyEvent);
                             }
                             settings.onLoad.call($tab[0], currentPath, parameterArray, historyEvent);
-                        } else if (tabPath.search('/') == -1 && tabPath !== '') {
+                        } else if (tabPath.search('/') === -1 && tabPath !== '') {
                             // look for in page anchor
                             tabPath = module.escape.string(tabPath);
                             $anchor = $('#' + tabPath + ', a[name="' + tabPath + '"]');
@@ -431,7 +431,7 @@
 
                 scrollTo: function ($element) {
                     var
-                        scrollOffset = ($element && $element.length > 0)
+                        scrollOffset = $element && $element.length > 0
                             ? $element.offset().top
                             : false
                     ;
@@ -447,10 +447,10 @@
                             $tab = module.get.tabElement(tabPath),
                             tab  = $tab[0]
                         ;
-                        evaluateScripts = (evaluateScripts !== undefined)
+                        evaluateScripts = evaluateScripts !== undefined
                             ? evaluateScripts
                             : settings.evaluateScripts;
-                        if (typeof settings.cacheType === 'string' && settings.cacheType.toLowerCase() == 'dom' && typeof html !== 'string') {
+                        if (typeof settings.cacheType === 'string' && settings.cacheType.toLowerCase() === 'dom' && typeof html !== 'string') {
                             $tab
                                 .empty()
                                 .append($(html).clone(true))
@@ -481,7 +481,7 @@
                                     'X-Remote': true,
                                 },
                                 onSuccess: function (response) {
-                                    if (settings.cacheType == 'response') {
+                                    if (settings.cacheType === 'response') {
                                         module.cache.add(fullTabPath, response);
                                     }
                                     module.update.content(tabPath, response);
@@ -496,7 +496,7 @@
 
                                     if (settings.loadOnce) {
                                         module.cache.add(fullTabPath, true);
-                                    } else if (typeof settings.cacheType === 'string' && settings.cacheType.toLowerCase() == 'dom' && $tab.children().length > 0) {
+                                    } else if (typeof settings.cacheType === 'string' && settings.cacheType.toLowerCase() === 'dom' && $tab.children().length > 0) {
                                         setTimeout(function () {
                                             var
                                                 $clone = $tab.children().clone(true)
@@ -513,7 +513,7 @@
                                 },
                             },
                             request         = $tab.api('get request') || false,
-                            existingRequest = (request && request.state() === 'pending'),
+                            existingRequest = request && request.state() === 'pending',
                             requestSettings,
                             cachedContent
                         ;
@@ -525,7 +525,7 @@
                             module.activate.tab(tabPath);
                             module.debug('Adding cached content', fullTabPath);
                             if (!settings.loadOnce) {
-                                if (settings.evaluateScripts == 'once') {
+                                if (settings.evaluateScripts === 'once') {
                                     module.update.content(tabPath, cachedContent, false);
                                 } else {
                                     module.update.content(tabPath, cachedContent);
@@ -554,7 +554,7 @@
                     tab: function (tabPath) {
                         var
                             $tab          = module.get.tabElement(tabPath),
-                            $deactiveTabs = (settings.deactivate == 'siblings')
+                            $deactiveTabs = settings.deactivate === 'siblings'
                                 ? $tab.siblings($tabs)
                                 : $tabs.not($tab),
                             isActive      = $tab.hasClass(className.active)
@@ -575,7 +575,7 @@
                     navigation: function (tabPath) {
                         var
                             $navigation         = module.get.navElement(tabPath),
-                            $deactiveNavigation = (settings.deactivate == 'siblings')
+                            $deactiveNavigation = settings.deactivate === 'siblings'
                                 ? $navigation.siblings($allModules)
                                 : $allModules.not($navigation),
                             isActive    = $navigation.hasClass(className.active)
@@ -611,8 +611,8 @@
 
                 is: {
                     tab: function (tabName) {
-                        return (tabName !== undefined)
-                            ? (module.get.tabElement(tabName).length > 0)
+                        return tabName !== undefined
+                            ? module.get.tabElement(tabName).length > 0
                             : false;
                     },
                 },
@@ -666,7 +666,7 @@
                         $fullPathTab = $tabs.filter('[data-' + metadata.tab + '="' + module.escape.string(tabPath) + '"]');
                         $simplePathTab = $tabs.filter('[data-' + metadata.tab + '="' + module.escape.string(lastTab) + '"]');
 
-                        return ($fullPathTab.length > 0)
+                        return $fullPathTab.length > 0
                             ? $fullPathTab
                             : $simplePathTab;
                     },
@@ -701,7 +701,7 @@
                 utilities: {
                     filterArray: function (keepArray, removeArray) {
                         return $.grep(keepArray, function (keepValue) {
-                            return ($.inArray(keepValue, removeArray) == -1);
+                            return $.inArray(keepValue, removeArray) === -1;
                         });
                     },
                     last: function (array) {
@@ -837,17 +837,17 @@
                         query = query.split(/[ .]/);
                         maxDepth = query.length - 1;
                         $.each(query, function (depth, value) {
-                            var camelCaseValue = (depth != maxDepth)
+                            var camelCaseValue = depth !== maxDepth
                                 ? value + query[depth + 1].charAt(0).toUpperCase() + query[depth + 1].slice(1)
                                 : query
                             ;
-                            if ($.isPlainObject(object[camelCaseValue]) && (depth != maxDepth)) {
+                            if ($.isPlainObject(object[camelCaseValue]) && (depth !== maxDepth)) {
                                 object = object[camelCaseValue];
                             } else if (object[camelCaseValue] !== undefined) {
                                 found = object[camelCaseValue];
 
                                 return false;
-                            } else if ($.isPlainObject(object[value]) && (depth != maxDepth)) {
+                            } else if ($.isPlainObject(object[value]) && (depth !== maxDepth)) {
                                 object = object[value];
                             } else if (object[value] !== undefined) {
                                 found = object[value];
@@ -889,7 +889,7 @@
             }
         });
 
-        return (returnedValue !== undefined)
+        return returnedValue !== undefined
             ? returnedValue
             : this;
     };

package/src/definitions/modules/toast.js

@@ -15,7 +15,7 @@
         return typeof obj === 'function' && typeof obj.nodeType !== 'number';
     }
 
-    window = (window !== undefined && window.Math === Math)
+    window = window !== undefined && window.Math === Math
         ? window
         : globalThis;
 
@@ -28,13 +28,13 @@
             performance    = [],
 
             query          = arguments[0],
-            methodInvoked  = (typeof query === 'string'),
+            methodInvoked  = typeof query === 'string',
             queryArguments = [].slice.call(arguments, 1),
             returnedValue
         ;
         $allModules.each(function () {
             var
-                settings          = ($.isPlainObject(parameters))
+                settings          = $.isPlainObject(parameters)
                     ? $.extend(true, {}, $.fn.toast.settings, parameters)
                     : $.extend({}, $.fn.toast.settings),
 
@@ -55,7 +55,7 @@
                 $progressBar,
                 $animationObject,
                 $close,
-                $context         = (settings.context)
+                $context         = settings.context
                     ? ([window, document].indexOf(settings.context) < 0 ? $(document).find(settings.context) : $(settings.context))
                     : $('body'),
 
@@ -147,7 +147,11 @@
                 create: {
                     container: function () {
                         module.verbose('Creating container');
-                        $context.append($('<div/>', { class: settings.position + ' ' + className.container + ' ' + (settings.horizontal ? className.horizontal : '') + ' ' + (settings.context && settings.context !== 'body' ? className.absolute : '') }));
+                        $context.append($('<div/>', {
+                            class: settings.position + ' ' + className.container + ' '
+                            + (settings.horizontal ? className.horizontal : '') + ' '
+                            + (settings.context && settings.context !== 'body' ? className.absolute : ''),
+                        }));
                     },
                     id: function () {
                         id = (Math.random().toString(16) + '000000000').slice(2, 10);
@@ -208,7 +212,7 @@
                         } else {
                             $toast = settings.cloneModule ? $module.clone().removeAttr('id') : $module;
                             $close = $toast.find('> i' + module.helpers.toClass(className.close));
-                            settings.closeIcon = ($close.length > 0);
+                            settings.closeIcon = $close.length > 0;
                             if (iconClass !== '') {
                                 $toast.find(selector.icon).attr('class', iconClass + ' ' + className.icon);
                             }
@@ -242,10 +246,15 @@
                             }
                             settings.actions.forEach(function (el) {
                                 var
-                                    icon = el[fields.icon] ? '<i ' + (el[fields.text] ? 'aria-hidden="true"' : '') + ' class="' + module.helpers.deQuote(el[fields.icon]) + ' icon"></i>' : '',
+                                    icon = el[fields.icon]
+                                        ? '<i ' + (el[fields.text] ? 'aria-hidden="true"' : '')
+                                            + ' class="' + module.helpers.deQuote(el[fields.icon]) + ' icon"></i>'
+                                        : '',
                                     text = module.helpers.escape(el[fields.text] || '', settings.preserveHTML),
                                     cls = module.helpers.deQuote(el[fields.class] || ''),
-                                    click = el[fields.click] && isFunction(el[fields.click]) ? el[fields.click] : function () {}
+                                    click = el[fields.click] && isFunction(el[fields.click])
+                                        ? el[fields.click]
+                                        : function () {}
                                 ;
                                 $actions.append($('<button/>', {
                                     html: icon + text,
@@ -482,7 +491,13 @@
                         return id;
                     },
                     containers: function () {
-                        return $context.children(module.helpers.toClass(settings.position) + selector.container + (settings.horizontal ? module.helpers.toClass(className.horizontal) : ':not(' + module.helpers.toClass(className.horizontal) + ')') + (settings.context && settings.context !== 'body' ? module.helpers.toClass(className.absolute) : ':not(' + module.helpers.toClass(className.absolute) + ')'));
+                        return $context.children(module.helpers.toClass(settings.position) + selector.container
+                            + (settings.horizontal
+                                ? module.helpers.toClass(className.horizontal)
+                                : ':not(' + module.helpers.toClass(className.horizontal) + ')')
+                            + (settings.context && settings.context !== 'body'
+                                ? module.helpers.toClass(className.absolute)
+                                : ':not(' + module.helpers.toClass(className.absolute) + ')'));
                     },
                     container: function () {
                         return module.get.containers()[0];
@@ -497,7 +512,11 @@
                         return $(module.get.container()).find(selector.box);
                     },
                     iconClass: function () {
-                        return typeof settings.showIcon === 'string' ? settings.showIcon : settings.showIcon && settings.icons[settings.class] ? settings.icons[settings.class] : '';
+                        return typeof settings.showIcon === 'string'
+                            ? settings.showIcon
+                            : (settings.showIcon && settings.icons[settings.class]
+                                ? settings.icons[settings.class]
+                                : '');
                     },
                     remainingTime: function () {
                         return $animationObject ? $animationObject.css('opacity') * settings.displayTime : 0;
@@ -715,17 +734,17 @@
                         query = query.split(/[ .]/);
                         maxDepth = query.length - 1;
                         $.each(query, function (depth, value) {
-                            var camelCaseValue = (depth != maxDepth)
+                            var camelCaseValue = depth !== maxDepth
                                 ? value + query[depth + 1].charAt(0).toUpperCase() + query[depth + 1].slice(1)
                                 : query
                             ;
-                            if ($.isPlainObject(object[camelCaseValue]) && (depth != maxDepth)) {
+                            if ($.isPlainObject(object[camelCaseValue]) && (depth !== maxDepth)) {
                                 object = object[camelCaseValue];
                             } else if (object[camelCaseValue] !== undefined) {
                                 found = object[camelCaseValue];
 
                                 return false;
-                            } else if ($.isPlainObject(object[value]) && (depth != maxDepth)) {
+                            } else if ($.isPlainObject(object[value]) && (depth !== maxDepth)) {
                                 object = object[value];
                             } else if (object[value] !== undefined) {
                                 found = object[value];
@@ -769,7 +788,7 @@
             }
         });
 
-        return (returnedValue !== undefined)
+        return returnedValue !== undefined
             ? returnedValue
             : this;
     };
@@ -913,16 +932,21 @@
                 return n1 * x * x;
             }
             if (x < 2 / d1) {
-                return n1 * (x -= 1.5 / d1) * x + 0.75;
+                x -= 1.5 / d1;
+
+                return n1 * x * x + 0.75;
             }
             if (x < 2.5 / d1) {
-                return n1 * (x -= 2.25 / d1) * x + 0.9375;
+                x -= 2.25 / d1;
+
+                return n1 * x * x + 0.9375;
             }
+            x -= 2.625 / d1;
 
-            return n1 * (x -= 2.625 / d1) * x + 0.984375;
+            return n1 * x * x + 0.984375;
         },
         easeOutCubic: function (t) {
-            return (--t) * t * t + 1;
+            return --t * t * t + 1;
         },
     });
 })(jQuery, window, document);

package/src/definitions/modules/transition.js

@@ -15,7 +15,7 @@
         return typeof obj === 'function' && typeof obj.nodeType !== 'number';
     }
 
-    window = (window !== undefined && window.Math === Math)
+    window = window !== undefined && window.Math === Math
         ? window
         : globalThis;
 
@@ -30,7 +30,7 @@
             moduleArguments = arguments,
             query           = moduleArguments[0],
             queryArguments  = [].slice.call(arguments, 1),
-            methodInvoked   = (typeof query === 'string'),
+            methodInvoked   = typeof query === 'string',
 
             returnedValue
         ;
@@ -139,11 +139,11 @@
                             ? module.get.direction()
                             : 'static';
                     }
-                    interval = (interval !== undefined)
+                    interval = interval !== undefined
                         ? interval
                         : settings.interval;
-                    shouldReverse = (settings.reverse == 'auto' && direction == className.outward);
-                    delay = (shouldReverse || settings.reverse === true)
+                    shouldReverse = settings.reverse === 'auto' && direction === className.outward;
+                    delay = shouldReverse || settings.reverse === true
                         ? ($allModules.length - index) * interval
                         : index * interval;
                     module.debug('Delaying animation by', delay);
@@ -253,7 +253,7 @@
                         var
                             style          = $module.attr('style'),
                             currentDisplay = $module.css('display'),
-                            emptyStyle     = (style === undefined || style === '')
+                            emptyStyle     = style === undefined || style === ''
                         ;
                         if (currentDisplay !== 'none' && !module.is.hidden()) {
                             module.verbose('Overriding default display to hide element');
@@ -315,7 +315,7 @@
                     },
                     duration: function (animationName, duration) {
                         duration = duration || settings.duration;
-                        duration = (typeof duration === 'number')
+                        duration = typeof duration === 'number'
                             ? duration + 'ms'
                             : duration;
                         if (duration || duration === 0) {
@@ -329,7 +329,7 @@
                     },
                     direction: function (direction) {
                         direction = direction || module.get.direction();
-                        if (direction == className.inward) {
+                        if (direction === className.inward) {
                             module.set.inward();
                         } else {
                             module.set.outward();
@@ -525,7 +525,7 @@
                     animationClass: function (animation) {
                         var
                             animationClass = animation || settings.animation,
-                            directionClass = (module.can.transition() && !module.has.direction())
+                            directionClass = module.can.transition() && !module.has.direction()
                                 ? module.get.direction() + ' '
                                 : ''
                         ;
@@ -536,7 +536,7 @@
                             + animationClass;
                     },
                     currentAnimation: function () {
-                        return (module.cache && module.cache.animation !== undefined)
+                        return module.cache && module.cache.animation !== undefined
                             ? module.cache.animation
                             : false;
                     },
@@ -579,14 +579,14 @@
                             duration = $module.css('animation-duration') || 0;
                         }
 
-                        return (typeof duration === 'string')
-                            ? (duration.indexOf('ms') > -1)
+                        return typeof duration === 'string'
+                            ? (duration.indexOf('ms') > -1
                                 ? parseFloat(duration)
-                                : parseFloat(duration) * 1000
+                                : parseFloat(duration) * 1000)
                             : duration;
                     },
                     displayType: function (shouldDetermine) {
-                        shouldDetermine = (shouldDetermine !== undefined)
+                        shouldDetermine = shouldDetermine !== undefined
                             ? shouldDetermine
                             : true;
                         if (settings.displayType) {
@@ -700,10 +700,10 @@
                             }
 
                             $clone.remove();
-                            if (currentAnimation != inAnimation) {
+                            if (currentAnimation !== inAnimation) {
                                 module.debug('Direction exists for animation', animation);
                                 directionExists = true;
-                            } else if (currentAnimation == 'none' || !currentAnimation) {
+                            } else if (currentAnimation === 'none' || !currentAnimation) {
                                 module.debug('No animation defined in css', animation);
 
                                 return;
@@ -714,13 +714,13 @@
                             module.save.transitionExists(animation, directionExists);
                         }
 
-                        return (transitionExists !== undefined)
+                        return transitionExists !== undefined
                             ? transitionExists
                             : directionExists;
                     },
                     animate: function () {
                         // can transition does not return a value if animation does not exist
-                        return (module.can.transition() !== undefined);
+                        return module.can.transition() !== undefined;
                     },
                 },
 
@@ -741,7 +741,7 @@
                         animation = animation || settings.animation;
                         animation = '.' + animation.replace(' ', '.');
 
-                        return ($module.filter(animation).length > 0);
+                        return $module.filter(animation).length > 0;
                     },
                     visible: function () {
                         return $module.is(':visible');
@@ -750,7 +750,7 @@
                         return $module.css('visibility') === 'hidden';
                     },
                     supported: function () {
-                        return (animationEnd !== false);
+                        return animationEnd !== false;
                     },
                 },
 
@@ -943,17 +943,17 @@
                         query = query.split(/[ .]/);
                         maxDepth = query.length - 1;
                         $.each(query, function (depth, value) {
-                            var camelCaseValue = (depth != maxDepth)
+                            var camelCaseValue = depth !== maxDepth
                                 ? value + query[depth + 1].charAt(0).toUpperCase() + query[depth + 1].slice(1)
                                 : query
                             ;
-                            if ($.isPlainObject(object[camelCaseValue]) && (depth != maxDepth)) {
+                            if ($.isPlainObject(object[camelCaseValue]) && (depth !== maxDepth)) {
                                 object = object[camelCaseValue];
                             } else if (object[camelCaseValue] !== undefined) {
                                 found = object[camelCaseValue];
 
                                 return false;
-                            } else if ($.isPlainObject(object[value]) && (depth != maxDepth)) {
+                            } else if ($.isPlainObject(object[value]) && (depth !== maxDepth)) {
                                 object = object[value];
                             } else if (object[value] !== undefined) {
                                 found = object[value];
@@ -978,7 +978,7 @@
                         returnedValue = response;
                     }
 
-                    return (found !== undefined)
+                    return found !== undefined
                         ? found
                         : false;
                 },
@@ -986,7 +986,7 @@
             module.initialize();
         });
 
-        return (returnedValue !== undefined)
+        return returnedValue !== undefined
             ? returnedValue
             : this;
     };

package/tasks/admin/components/create.js

@@ -14,7 +14,7 @@
   * create meteor.js file
 */
 
-let
+const
     gulp            = require('gulp'),
 
     // node dependencies

package/tasks/admin/components/init.js

@@ -12,7 +12,7 @@
 
 */
 
-let
+const
     gulp      = require('gulp'),
 
     // node dependencies
@@ -85,7 +85,7 @@ module.exports = function (callback) {
         }
 
         // clean folder
-        if (release.outputRoot.search('../repos') == 0) {
+        if (release.outputRoot.startsWith('../repos')) {
             console.info('Cleaning dir', outputDirectory);
             del.sync([outputDirectory + '**/*'], { silent: true, force: true });
         }

package/tasks/admin/components/update.js

@@ -12,7 +12,7 @@
 
 */
 
-let
+const
     gulp           = require('gulp'),
 
     // node dependencies
@@ -68,7 +68,7 @@ module.exports = function (callback) {
             gitURL               = 'https://github.com/' + release.org + '/' + repoName + '.git',
             repoURL              = 'https://github.com/' + release.org + '/' + repoName + '/',
 
-            commitArgs = (oAuth.name !== undefined && oAuth.email !== undefined)
+            commitArgs = oAuth.name !== undefined && oAuth.email !== undefined
                 ? '--author "' + oAuth.name + ' <' + oAuth.email + '>"'
                 : '',
 
@@ -76,9 +76,9 @@ module.exports = function (callback) {
                 ? require(outputDirectory + 'package.json') // eslint-disable-line import/no-dynamic-require
                 : false,
 
-            isNewVersion  = (version && componentPackage.version != version),
+            isNewVersion  = version && componentPackage.version !== version,
 
-            commitMessage = (isNewVersion)
+            commitMessage = isNewVersion
                 ? 'Updated component to version ' + version
                 : 'Updated files from main repo',