From 6f202ef85710374a832cb38701302d999b18fe5f Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Wed, 12 Apr 2017 13:37:11 -0700
Subject: [PATCH] tools: remove assert.fail() lint rule
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

assert.fail() has been updated to accept a single argument so that is no
longer an error. Remove lint rule that checks for assert.fail() with a
single argument.

PR-URL: https://github.com/nodejs/node/pull/12293
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 .eslintrc.yaml | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/.eslintrc.yaml b/.eslintrc.yaml
index 9190475e9b..630fd19688 100644
--- a/.eslintrc.yaml
+++ b/.eslintrc.yaml
@@ -110,9 +110,6 @@ rules:
   }, {
     selector: "ThrowStatement > CallExpression[callee.name=/Error$/]",
     message: "Use new keyword when throwing an Error."
-    }, {
-    selector: "CallExpression[callee.object.name='assert'][callee.property.name='fail'][arguments.length=1]",
-    message: "assert.fail() message should be third argument"
     }]
   no-tabs: 2
   no-trailing-spaces: 2