[gjs: 2/43] CI: Use new eslint syntax for globals



commit 19fb99633ad318e267723b12b731b8ab7a32917c
Author: Philip Chimento <philip chimento gmail com>
Date:   Tue Aug 13 21:07:54 2019 -0700

    CI: Use new eslint syntax for globals
    
    Instead of 'true' and 'false' eslint now recommends using 'readonly',
    'writable', or 'off'. We make the globals readonly, except for the
    timeout functions in tests (because we overwrite those in the first
    place to make Jasmine work.)

 .eslintrc.yml                    | 20 ++++++++++----------
 installed-tests/js/.eslintrc.yml |  8 ++++----
 2 files changed, 14 insertions(+), 14 deletions(-)
---
diff --git a/.eslintrc.yml b/.eslintrc.yml
index 26216fed..50da632d 100644
--- a/.eslintrc.yml
+++ b/.eslintrc.yml
@@ -74,15 +74,15 @@ rules:
     - error
     - int32Hint: false
 globals:
-  ARGV: false
-  Debugger: false
-  GIRepositoryGType: false
-  imports: false
-  Intl: false
-  log: false
-  logError: false
-  print: false
-  printerr: false
-  window: false
+  ARGV: readonly
+  Debugger: readonly
+  GIRepositoryGType: readonly
+  imports: readonly
+  Intl: readonly
+  log: readonly
+  logError: readonly
+  print: readonly
+  printerr: readonly
+  window: readonly
 parserOptions:
   ecmaVersion: 2017
diff --git a/installed-tests/js/.eslintrc.yml b/installed-tests/js/.eslintrc.yml
index 72c5896f..b91f167b 100644
--- a/installed-tests/js/.eslintrc.yml
+++ b/installed-tests/js/.eslintrc.yml
@@ -2,7 +2,7 @@
 env:
   jasmine: true
 globals:
-  clearInterval: false
-  clearTimeout: false
-  setInterval: false
-  setTimeout: false
+  clearInterval: writable
+  clearTimeout: writable
+  setInterval: writable
+  setTimeout: writable


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]