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 <[email protected]>
Gerrit-Reviewer: Esanders <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: Trevor Parscal <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
