Commit f0409be
tools: lint for use of internalBinding()
Use of process.binding() has largely been replaced by
internalBinding(). This commit updates the custom crypto
check ESLint rule to check for both process.binding() and
internalBinding().
Refs: #24952
PR-URL: #25395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>1 parent 3db7a9f commit f0409beCopy full SHA for f0409be
File tree
Expand file treeCollapse file tree
2 files changed
+25
-6
lines changedOpen diff view settings
Filter options
- test/parallel
- tools/eslint-rules
Expand file treeCollapse file tree
2 files changed
+25
-6
lines changedOpen diff view settings
Collapse file
test/parallel/test-eslint-crypto-check.js
Copy file name to clipboardExpand all lines: test/parallel/test-eslint-crypto-check.js+18Lines changed: 18 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
19 | 19 | |
20 | 20 | |
21 | 21 | |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
22 | 28 | |
23 | 29 | |
24 | 30 | |
| ||
51 | 57 | |
52 | 58 | |
53 | 59 | |
| 60 | + |
| 61 | + |
| 62 | + |
| 63 | + |
| 64 | + |
| 65 | + |
| 66 | + |
| 67 | + |
| 68 | + |
| 69 | + |
| 70 | + |
| 71 | + |
54 | 72 | |
55 | 73 | |
56 | 74 | |
Collapse file
tools/eslint-rules/rules-utils.js
Copy file name to clipboardExpand all lines: tools/eslint-rules/rules-utils.js+7-6Lines changed: 7 additions & 6 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
33 | 33 | |
34 | 34 | |
35 | 35 | |
36 | | - |
| 36 | + |
37 | 37 | |
38 | 38 | |
39 | | - |
40 | | - |
41 | | - |
42 | | - |
43 | | - |
| 39 | + |
| 40 | + |
| 41 | + |
| 42 | + |
| 43 | + |
| 44 | + |
44 | 45 | |
45 | 46 | |
46 | 47 | |
|
0 commit comments