FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

tools,test: enable no-prototype-builtins · nodejs/node@6612056 · GitHub

/ node Public

Commit 6612056

Browse files
Trott
authored and
danielleadams
committed
tools,test: enable no-prototype-builtins
PR-URL: #41801 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com>
1 parent 673c1fd commit 6612056

2 files changed

Lines changed: 1 addition & 1 deletion

File tree

‎.eslintrc.js‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -297,7 +297,6 @@ module.exports = {
297297
// ESLint recommended rules that we disable
298298
'no-empty': 'off',
299299
'no-inner-declarations': 'off',
300-
'no-prototype-builtins': 'off',
301300

302301
// JSDoc recommended rules that we disable
303302
'jsdoc/require-jsdoc': 'off',

‎test/js-native-api/test_object/test_null.js‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ const expectedForProperty = {
1414
assert.deepStrictEqual(testNull.setProperty(), expectedForProperty);
1515
assert.deepStrictEqual(testNull.getProperty(), expectedForProperty);
1616
assert.deepStrictEqual(testNull.hasProperty(), expectedForProperty);
17+
// eslint-disable-next-line no-prototype-builtins
1718
assert.deepStrictEqual(testNull.hasOwnProperty(), expectedForProperty);
1819
// It's OK not to want the result of a deletion.
1920
assert.deepStrictEqual(testNull.deleteProperty(),

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL