Commit c217b43
assert, tools: enforce strict (not)equal in eslint
Extend no-restricted-properties to catch use of assert.equal() and
assert.notEqual() and require assert.strictEqual() or
assert.notStrictEqual() instead.
Also update the eslint-ignore in lib/assert.js to avoid
assert.equal/notEqual linter errors in their definitions.
PR-URL: #10698
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>1 parent 51f4c8b commit c217b43Copy full SHA for c217b43
File tree
Expand file treeCollapse file tree
2 files changed
+17
-14
lines changedOpen diff view settings
Filter options
- lib
Expand file treeCollapse file tree
2 files changed
+17
-14
lines changedOpen diff view settings
Collapse file
+16-12Lines changed: 16 additions & 12 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
62 | 62 | |
63 | 63 | |
64 | 64 | |
65 | | - |
66 | | - |
67 | | - |
68 | | - |
69 | | - |
70 | | - |
71 | | - |
72 | | - |
73 | | - |
74 | | - |
75 | | - |
| 65 | + |
| 66 | + |
| 67 | + |
| 68 | + |
| 69 | + |
| 70 | + |
| 71 | + |
| 72 | + |
| 73 | + |
| 74 | + |
| 75 | + |
| 76 | + |
| 77 | + |
| 78 | + |
| 79 | + |
76 | 80 | |
77 | 81 | |
78 | 82 | |
| ||
86 | 90 | |
87 | 91 | |
88 | 92 | |
89 | | - |
| 93 | + |
90 | 94 | |
91 | 95 | |
92 | 96 | |
|
Collapse file
+1-2Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
109 | 109 | |
110 | 110 | |
111 | 111 | |
112 | | - |
| 112 | + |
113 | 113 | |
114 | 114 | |
115 | 115 | |
| ||
127 | 127 | |
128 | 128 | |
129 | 129 | |
130 | | - |
131 | 130 | |
132 | 131 | |
133 | 132 | |
|
0 commit comments