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

tools: avoid parsing test files twice · nodejs/node@6197eee · GitHub

/ node Public

Commit 6197eee

Browse files
pmarchini
authored and
richardlau
committed
tools: avoid parsing test files twice
PR-URL: #59526 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
1 parent 36dd856 commit 6197eee

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

‎tools/test.py‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -779,6 +779,11 @@ def Execute(args, context, timeout=None, env=None, disable_core_files=False,
779779
for key, value in env.items():
780780
env_copy[key] = value
781781

782+
# We append NODE_SKIP_FLAG_CHECK (ref: test/common/index.js)
783+
# to avoid parsing the test files twice when looking for
784+
# flags or environment variables defined via // Flags: and // Env:
785+
env_copy["NODE_SKIP_FLAG_CHECK"] = "true"
786+
782787
preexec_fn = None
783788

784789
def disableCoreFiles():

0 commit comments

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