Fix Rubocop offenses, improve SQL duration format and changelog entry

Signed-off-by: Rémy Coutable <remy@rymai.me>

2 files changed, 14 insertions, 18 deletions

diff --git a/app/assets/javascripts/peek.js b/app/assets/javascripts/peek.js
index 06a07c19e09..60f8bdd14c0 100644
--- a/app/assets/javascripts/peek.js
+++ b/app/assets/javascripts/peek.js
@@ -1,19 +1,16 @@
 import 'vendor/peek';
 import 'vendor/peek.performance_bar';
 
-(function() {
-  $(document).on('click', '#peek-show-queries', function(e) {
-    var $modal;
-    $('.peek-rblineprof-modal').hide();
-    $modal = $('#modal-peek-pg-queries');
-    if ($modal.length) {
-      $modal.modal('toggle');
-    }
-    return e.preventDefault();
-  });
+$(document).on('click', '#peek-show-queries', function(e) {
+  e.preventDefault();
+  $('.peek-rblineprof-modal').hide();
+  let $modal = $('#modal-peek-pg-queries');
+  if ($modal.length) {
+    $modal.modal('toggle');
+  }
+});
 
-  $(document).on('click', '.js-lineprof-file', function(e) {
-    $(this).parents('.heading').next('div').toggle();
-    return e.preventDefault();
-  });
-}).call(window);
+$(document).on('click', '.js-lineprof-file', function(e) {
+  e.preventDefault();
+  $(this).parents('.heading').next('div').toggle();
+});
diff --git a/app/assets/javascripts/shortcuts.js b/app/assets/javascripts/shortcuts.js
index 2c7698eb174..09cb914e884 100644
--- a/app/assets/javascripts/shortcuts.js
+++ b/app/assets/javascripts/shortcuts.js
@@ -60,11 +60,10 @@ import findAndFollowLink from './shortcuts_dashboard_navigation';
       e.preventDefault();
       if (Cookies.get('perf_bar_enabled') === 'true') {
         Cookies.remove('perf_bar_enabled', { path: '/' });
-      }
-      else {
+      } else {
         Cookies.set('perf_bar_enabled', true, { path: '/' });
       }
-      return gl.utils.refreshCurrentPage();
+      gl.utils.refreshCurrentPage();
     };
 
     Shortcuts.prototype.toggleMarkdownPreview = function(e) {