jenkins-bot has submitted this change and it was merged.

Change subject: build: Update jscs and jshint config
......................................................................


build: Update jscs and jshint config

jshint:
* Remove coding style option "curly" (already covered by jscs).
  Follows-up 149bec2 which added jscs/requireCurlyBraces.

jscs:
* Remove rule for "disallowRightStickedOperators"
  The disallowRightStickedOperators rule is obsolete with
  requireSpace{Before,After}BinaryOperators and fixes a bug where
  the following would be considered invalid when it shouldn't:

      +!!constructor.static.matchFunction
      // Operator + should not stick to following expression

   This is a false positive due to the tricky nature of
   disallowRightStickedOperators and for that reason this rule
   has been deprecated and scheduled for removal in jscs 2.0.0

Change-Id: Ib731f11b1184d0e9865104a065421446a78dbfec
---
M .jscsrc
M .jshintrc
2 files changed, 0 insertions(+), 17 deletions(-)

Approvals:
  Krinkle: Looks good to me, but someone else must approve
  Esanders: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/.jscsrc b/.jscsrc
index eb2c768..bc49e03 100644
--- a/.jscsrc
+++ b/.jscsrc
@@ -44,22 +44,6 @@
                "<="
        ],
        "requireRightStickedOperators": ["!"],
-       "disallowRightStickedOperators": [
-               "?",
-               "+",
-               "/",
-               "*",
-               ":",
-               "=",
-               "==",
-               "===",
-               "!=",
-               "!==",
-               ">",
-               ">=",
-               "<",
-               "<="
-    ],
        "requireLeftStickedOperators": [","],
        "disallowSpaceAfterPrefixUnaryOperators": [
                "++",
diff --git a/.jshintrc b/.jshintrc
index 40e5b98..7f73166 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -1,7 +1,6 @@
 {
        // Enforcing
        "bitwise": true,
-       "curly": true,
        "eqeqeq": true,
        "freeze": true,
        "latedef": true,

-- 
To view, visit https://gerrit.wikimedia.org/r/133271
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib731f11b1184d0e9865104a065421446a78dbfec
Gerrit-PatchSet: 2
Gerrit-Project: oojs/core
Gerrit-Branch: master
Gerrit-Owner: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: Esanders <esand...@wikimedia.org>
Gerrit-Reviewer: Jforrester <jforres...@wikimedia.org>
Gerrit-Reviewer: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: Trevor Parscal <tpars...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to