Skip to content

Commit

Permalink
debugger: remove final lint exceptions in inspect_repl.js
Browse files Browse the repository at this point in the history
Adding a return when it's not really a getter is kind of misleading, but
so is using a getter for something that doesn't return anything, so
¯\_(ツ)_/¯.

PR-URL: #39078
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
  • Loading branch information
Trott authored and targos committed Sep 4, 2021
1 parent 8705263 commit e5826ab
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions lib/internal/debugger/inspect_repl.js
@@ -1,6 +1,3 @@
// TODO(trott): enable ESLint
/* eslint-disable getter-return */

'use strict';

const {
Expand Down Expand Up @@ -899,7 +896,7 @@ function createRepl(inspector) {

copyOwnProperties(context, {
get help() {
print(HELP);
return print(HELP);
},

get run() {
Expand Down Expand Up @@ -1078,7 +1075,7 @@ function createRepl(inspector) {
repl.setPrompt('> ');

print('Press Ctrl+C to leave debug repl');
repl.displayPrompt();
return repl.displayPrompt();
},

get version() {
Expand Down

0 comments on commit e5826ab

Please sign in to comment.