helloadhavan/github_issues huggingface dataset:
Repository: huggingface/transformers
Pull request: fix asr pipeline ut failures

4 ASR cases failed on transformers

> pytest -rA tests/pipelines/test_pipelines_automatic_speech_recognition.py::AutomaticSpeechRecognitionPipelineTests::test_pipeline_assisted_generation
> pytest -rA tests/test_pipeline_mixin.py::AutomaticSpeechRecognitionPipelineTests::test_pipeline_assisted_generation
> pytest -rA tests/pipelines/test_pipelines_automatic_speech_recognition.py::AutomaticSpeechRecognitionPipelineTests::test_torch_whisper_batched
> pytest -rA tests/test_pipeline_mixin.py::AutomaticSpeechRecognitionPipelineTests::test_torch_whisper_batched


@SunMarc @ydshieh , pls help review, thx very much

Changed files:
filename: tests/pipelines/test_pipelines_automatic_speech_recognition.py
patch: @@ -615,7 +615,8 @@ def test_torch_whisper_batched(self):
             {"text": " Nor is Mr. Quilters' manner less interesting than his matter."},
         ]
 
-        output = speech_recognizer(ds["audio"], batch_size=2)
+        audio_arrays = [x.get_all_samples().data for x in ds["audio"]]
+        output = speech_recognizer(audio_arrays, batch_size=2)
         self.assertEqual(output, EXPECTED_OUTPUT)
 
     @slow
@@ -1763,11 +1764,11 @@ def test_pipeline_assisted_generation(self):
         pipe = pipeline("automatic-speech-recognition", model=model, assistant_model=model)
 
         # We can run the pipeline
-        prompt = load_dataset("hf-internal-testing/librispeech_asr_dummy", "clean", split="validation[:1]")["audio"]
-        _ = pipe(prompt)
+        prompt = load_dataset("hf-internal-testing/librispeech_asr_dummy", "clean", split="validation[:1]")[0]["audio"]
+        _ = pipe(prompt, generate_kwargs={"num_beams": 1})
 
         # It is running assisted generation under the hood (e.g. flags incompatible with assisted gen will crash)
-        with self.assertRaises(ValueError):
+        with self.assertRaises(TypeError):
             _ = pipe(prompt, generate_kwargs={"num_beams": 2})
 
     @require_torch
Repository: ollama/ollama
Pull request: Community Integration: tlm - cli copilot with ollama

I have been advised to create a PR to include [tlm](https://github.com/yusufcanb/tlm) inside [README.md](https://github.com/ollama/ollama/blob/main/README.md) during the KubeCon 2024 Paris by the Ollama staff. Thanks for all of them who expressed their excitement for what I've created. ❤

So, here is the PR to include [tlm](https://github.com/yusufcanb/tlm)  in Terminal section. Thanks again for the advise.


Changed files:
filename: README.md
patch: @@ -298,6 +298,7 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [tenere](https://github.com/pythops/tenere)
 - [llm-ollama](https://github.com/taketwo/llm-ollama) for [Datasette's LLM CLI](https://llm.datasette.io/en/stable/).
 - [ShellOracle](https://github.com/djcopley/ShellOracle)
+- [tlm](https://github.com/yusufcanb/tlm)
 
 ### Database
Repository: ollama/ollama
Pull request: Update README.md

Added Ollama Basic chat based on hyperdiv

Changed files:
filename: README.md
patch: @@ -280,7 +280,8 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [NextChat](https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web) with [Get Started Doc](https://docs.nextchat.dev/models/ollama)
 - [Odin Runes](https://github.com/leonid20000/OdinRunes)
 - [LLM-X: Progressive Web App](https://github.com/mrdjohnson/llm-x)
-- [Ollama-chats: (https://github.com/drazdra/ollama-chats)] - one-file browser chat rpg with Ollama
+- [Ollama Basic Chat: Uses HyperDiv Reactive UI] (https://github.com/rapidarchitect/ollama_basic_chat)
+- [Ollama-chats RPG: (https://github.com/drazdra/ollama-chats)]
 
 ### Terminal
Repository: facebook/react
Pull request: [compiler] repro of false positive for ValidatePreserveManualMemo

Stack from [ghstack](https://github.com/ezyang/ghstack) (oldest at bottom):
* #30428
* __->__ #30431
* #30399
* #30398
* #30397



Changed files:
filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/preserve-memo-validation/error.false-positive-useMemo-with-refs.flow.expect.md
patch: @@ -0,0 +1,38 @@
+
+## Input
+
+```javascript
+// @flow @validatePreserveExistingMemoizationGuarantees
+import { identity } from "shared-runtime";
+
+component Component(
+  disableLocalRef,
+  ref,
+) {
+  const localRef = useFooRef();
+  const mergedRef = useMemo(() => {
+    return disableLocalRef ? ref : identity(ref, localRef);
+  }, [disableLocalRef, ref, localRef]);
+  return <div ref={mergedRef} />;
+}
+
+```
+
+
+## Error
+
+```
+   7 | ) {
+   8 |   const localRef = useFooRef();
+>  9 |   const mergedRef = useMemo(() => {
+     |                             ^^^^^^^
+> 10 |     return disableLocalRef ? ref : identity(ref, localRef);
+     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+> 11 |   }, [disableLocalRef, ref, localRef]);
+     | ^^^^ CannotPreserveMemoization: React Compiler has skipped optimizing this component because the existing manual memoization could not be preserved. This value may be mutated later, which could cause the value to change unexpectedly (9:11)
+  12 |   return <div ref={mergedRef} />;
+  13 | }
+  14 |
+```
+          
+      
\ No newline at end of file

filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/preserve-memo-validation/error.false-positive-useMemo-with-refs.flow.js
patch: @@ -0,0 +1,13 @@
+// @flow @validatePreserveExistingMemoizationGuarantees
+import { identity } from "shared-runtime";
+
+component Component(
+  disableLocalRef,
+  ref,
+) {
+  const localRef = useFooRef();
+  const mergedRef = useMemo(() => {
+    return disableLocalRef ? ref : identity(ref, localRef);
+  }, [disableLocalRef, ref, localRef]);
+  return <div ref={mergedRef} />;
+}
Repository: nodejs/node
Pull request: test: fix `test-setproctitle` status when `ps` is not available

The exit code on Ubuntu is 127, the test is only checking for exit code 1.

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: test/parallel/test-setproctitle.js
patch: @@ -4,7 +4,7 @@ const common = require('../common');
 const { isMainThread } = require('worker_threads');
 
 // FIXME add sunos support
-if (common.isSunOS || common.isIBMi) {
+if (common.isSunOS || common.isIBMi || common.isWindows) {
   common.skip(`Unsupported platform [${process.platform}]`);
 }
 
@@ -25,15 +25,10 @@ assert.notStrictEqual(process.title, title);
 process.title = title;
 assert.strictEqual(process.title, title);
 
-// Test setting the title but do not try to run `ps` on Windows.
-if (common.isWindows) {
-  common.skip('Windows does not have "ps" utility');
-}
-
 try {
   execSync('command -v ps');
 } catch (err) {
-  if (err.status === 1) {
+  if (err.status === 1 || err.status === 127) {
     common.skip('The "ps" utility is not available');
   }
   throw err;
@@ -53,5 +48,5 @@ exec(cmd, common.mustSucceed((stdout, stderr) => {
     title += ` (${path.basename(process.execPath)})`;
 
   // Omitting trailing whitespace and \n
-  assert.strictEqual(stdout.replace(/\s+$/, '').endsWith(title), true);
+  assert.ok(stdout.trimEnd().endsWith(title));
 }));
Repository: electron/electron
Pull request: fix: calculate a hash for the Tag property of ToastNotification.

#### Description of Change

We encountered an issue on Windows, the notification won't popup when use web notification.
After our investigation. We found that when the URL exceeds a certain length, it will not popup the notification.
And it was caused by this change: https://github.com/electron/electron/pull/40242
In fact, the length of notification_id() will be affected by url, NotificationOptions, etc.

My solution is to calculate the hash of the notification_id() value.

Closes https://github.com/electron/electron/issues/42233
Closes https://github.com/electron/electron/issues/40433

#### Release Notes
Notes: none.


Changed files:
filename: shell/browser/notifications/win/windows_toast_notification.cc
patch: @@ -14,6 +14,7 @@
 #include <wrl\wrappers\corewrappers.h>
 
 #include "base/environment.h"
+#include "base/hash/hash.h"
 #include "base/logging.h"
 #include "base/strings/strcat.h"
 #include "base/strings/string_number_conversions.h"
@@ -71,6 +72,10 @@ void DebugLog(std::string_view log_msg) {
     LOG(INFO) << log_msg;
 }
 
+std::wstring GetTag(const std::string& notification_id) {
+  return base::NumberToWString(base::Hash(notification_id));
+}
+
 }  // namespace
 
 // static
@@ -146,7 +151,7 @@ void WindowsToastNotification::Remove() {
     return;
 
   ScopedHString group(kGroup);
-  ScopedHString tag(base::as_wcstr(base::UTF8ToUTF16(notification_id())));
+  ScopedHString tag(GetTag(notification_id()));
   notification_history->RemoveGroupedTagWithId(tag, group, app_id);
 }
 
@@ -199,7 +204,7 @@ HRESULT WindowsToastNotification::ShowInternal(
   REPORT_AND_RETURN_IF_FAILED(toast2->put_Group(group),
                               "WinAPI: Setting group failed");
 
-  ScopedHString tag(base::as_wcstr(base::UTF8ToUTF16(notification_id())));
+  ScopedHString tag(GetTag(notification_id()));
   REPORT_AND_RETURN_IF_FAILED(toast2->put_Tag(tag),
                               "WinAPI: Setting tag failed");
Repository: huggingface/transformers
Pull request: Remove some previous team members from allow list of triggering Github Actions

# What does this PR do?

Better security practice.

Changed files:
filename: .github/workflows/pr_build_doc_with_comment.yml
patch: @@ -14,7 +14,7 @@ permissions: {}
 jobs:
   get-pr-number:
     name: Get PR number
-    if: ${{ github.event.issue.state == 'open' && contains(fromJSON('["ydshieh", "ArthurZucker", "zucchini-nlp", "molbap", "gante", "LysandreJik", "Cyrilvallez", "Rocketknight1", "SunMarc", "muellerzr", "eustlb", "MekkCyber", "manueldeprada", "vasqu", "ivarflakstad", "stevhliu", "ebezzam", "itazap"]'), github.actor) && (startsWith(github.event.comment.body, 'build-doc')) }}
+    if: ${{ github.event.issue.state == 'open' && contains(fromJSON('["ydshieh", "ArthurZucker", "zucchini-nlp", "molbap", "gante", "LysandreJik", "Cyrilvallez", "Rocketknight1", "SunMarc", "eustlb", "MekkCyber", "vasqu", "ivarflakstad", "stevhliu", "ebezzam", "itazap"]'), github.actor) && (startsWith(github.event.comment.body, 'build-doc')) }}
     uses: ./.github/workflows/get-pr-number.yml
 
   get-pr-info:

filename: .github/workflows/self-comment-ci.yml
patch: @@ -29,7 +29,7 @@ jobs:
     runs-on: ubuntu-22.04
     name: Get PR number
     # For security: only allow team members to run
-    if: ${{ github.event.issue.state == 'open' && contains(fromJSON('["ydshieh", "ArthurZucker", "zucchini-nlp", "molbap", "gante", "LysandreJik", "Cyrilvallez", "Rocketknight1", "SunMarc", "muellerzr", "eustlb", "MekkCyber", "manueldeprada", "vasqu", "ivarflakstad", "stevhliu", "ebezzam", "remi-or", "itazap"]'), github.actor) && (startsWith(github.event.comment.body, 'run-slow') || startsWith(github.event.comment.body, 'run slow') || startsWith(github.event.comment.body, 'run_slow')) }}
+    if: ${{ github.event.issue.state == 'open' && contains(fromJSON('["ydshieh", "ArthurZucker", "zucchini-nlp", "molbap", "gante", "LysandreJik", "Cyrilvallez", "Rocketknight1", "SunMarc", "eustlb", "MekkCyber", "vasqu", "ivarflakstad", "stevhliu", "ebezzam", "remi-or", "itazap"]'), github.actor) && (startsWith(github.event.comment.body, 'run-slow') || startsWith(github.event.comment.body, 'run slow') || startsWith(github.event.comment.body, 'run_slow')) }}
     outputs:
       PR_NUMBER: ${{ steps.set_pr_number.outputs.PR_NUMBER }}
     steps:

filename: utils/process_bad_commit_report.py
patch: @@ -46,7 +46,6 @@
         "itazap",
         "ivarflakstad",
         "molbap",
-        "muellerzr",
         "remi-or",
         "stevhliu",
         "vasqu",
Repository: ollama/ollama
Pull request: doc: specify ADAPTER is optional


Changed files:
filename: docs/modelfile.md
patch: @@ -183,7 +183,7 @@ SYSTEM """<system message>"""
 
 ### ADAPTER
 
-The `ADAPTER` instruction specifies the LoRA adapter to apply to the base model. The value of this instruction should be an absolute path or a path relative to the Modelfile and the file must be in a GGML file format. The adapter should be tuned from the base model otherwise the behaviour is undefined.
+The `ADAPTER` instruction is an optional instruction that specifies any LoRA adapter that should apply to the base model. The value of this instruction should be an absolute path or a path relative to the Modelfile and the file must be in a GGML file format. The adapter should be tuned from the base model otherwise the behaviour is undefined.
 
 ```modelfile
 ADAPTER ./ollama-lora.bin
Repository: ollama/ollama
Pull request: llm: prevent race appending to slice

llm: prevent race appending to slice        

Previously, multiple goroutines were appending to the same unguarded
slice.

Also, convert slice declaration to idiomatic zero value form.

Also, convert errgroup.Group declaration to idiomatic zero value form.

Changed files:
filename: llm/payload_common.go
patch: @@ -11,6 +11,7 @@ import (
 	"path/filepath"
 	"runtime"
 	"strings"
+	"sync"
 
 	"golang.org/x/exp/slices"
 	"golang.org/x/sync/errgroup"
@@ -147,9 +148,10 @@ func extractDynamicLibs(payloadsDir, glob string) ([]string, error) {
 	if err != nil || len(files) == 0 {
 		return nil, payloadMissing
 	}
-	libs := []string{}
 
-	g := new(errgroup.Group)
+	var mu sync.Mutex
+	var libs []string
+	var g errgroup.Group
 	for _, file := range files {
 		pathComps := strings.Split(file, "/")
 		if len(pathComps) != pathComponentCount {
@@ -182,7 +184,9 @@ func extractDynamicLibs(payloadsDir, glob string) ([]string, error) {
 
 			destFile := filepath.Join(targetDir, filepath.Base(filename))
 			if strings.Contains(destFile, "server") {
+				mu.Lock()
 				libs = append(libs, destFile)
+				mu.Unlock()
 			}
 
 			destFp, err := os.OpenFile(destFile, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0o755)
Repository: nodejs/node
Pull request: test: lazy-load internalTTy

The documentation states the following:

```
### `NODE_REGENERATE_SNAPSHOTS`

If set, test snapshots for the current test are regenerated.
For example, `NODE_REGENERATE_SNAPSHOTS=1 out/Release/node test/parallel/test-runner-output.mjs`
will update all the test runner output snapshots.
```

However, due to an internal dependency, the suggested command fails:

![Image](https://github.com/user-attachments/assets/488856ca-f9cf-4179-940a-ec8c8d0c7776)

To avoid this error, I lazy-loaded the internal dependency:

![Image](https://github.com/user-attachments/assets/685da210-7d45-4dd7-a912-fac5a8af2437)


Changed files:
filename: test/parallel/test-runner-output.mjs
patch: @@ -6,13 +6,13 @@ import { describe, it } from 'node:test';
 import { hostname } from 'node:os';
 import { chdir, cwd } from 'node:process';
 import { fileURLToPath } from 'node:url';
-import internalTTy from 'internal/tty';
 
 const skipForceColors =
   process.config.variables.icu_gyp_path !== 'tools/icu/icu-generic.gyp' ||
   process.config.variables.node_shared_openssl;
 
-const canColorize = internalTTy.getColorDepth() > 2;
+// We're using dynamic import here to not break `NODE_REGENERATE_SNAPSHOTS`.
+const canColorize = (await import('internal/tty')).default.getColorDepth() > 2;
 const skipCoverageColors = !canColorize;
 
 function replaceTestDuration(str) {
Repository: vercel/next.js
Pull request: Update Rspack production test manifest

This auto-generated PR updates the production integration test manifest used when testing Rspack.

Changed files:
filename: test/rspack-build-tests-manifest.json
patch: @@ -3253,6 +3253,17 @@
     "flakey": [],
     "runtimeError": false
   },
+  "test/e2e/app-dir/log-file/log-file.test.ts": {
+    "passed": [
+      "log-file should capture RSC logging in log file",
+      "log-file should capture client logging in log file",
+      "log-file should capture logging in pages router"
+    ],
+    "failed": [],
+    "pending": [],
+    "flakey": [],
+    "runtimeError": false
+  },
   "test/e2e/app-dir/logging/fetch-logging.test.ts": {
     "passed": [
       "app-dir - fetch logging placeholder to satisfy at least one test when isNextDev is false",
@@ -21207,15 +21218,6 @@
     "flakey": [],
     "runtimeError": false
   },
-  "test/production/standalone-mode/node-modules/node-modules.test.ts": {
-    "passed": [
-      "standalone mode - NFT in node_modules should not trace process.cwd calls in node_modules"
-    ],
-    "failed": [],
-    "pending": [],
-    "flakey": [],
-    "runtimeError": false
-  },
   "test/production/standalone-mode/optimizecss/index.test.ts": {
     "passed": ["standalone mode and optimizeCss should work"],
     "failed": [],
@@ -21436,6 +21438,15 @@
     "flakey": [],
     "runtimeError": false
   },
+  "test/production/standalone-mode/tracing/tracing.test.ts": {
+    "passed": [
+      "standalone mode - tracing should trace process.cwd calls in node_modules"
+    ],
+    "failed": [],
+    "pending": [],
+    "flakey": [],
+    "runtimeError": false
+  },
   "test/production/standalone-mode/type-module/index.test.ts": {
     "passed": ["type-module should work"],
     "failed": [],
Repository: facebook/react
Pull request: [compiler] Repro for interaction b/w outlining and idx

Stack from [ghstack](https://github.com/ezyang/ghstack) (oldest at bottom):
* __->__ #30435
* #30434

babel-plugin-idx enforces that its 2nd argument is an arrow function, so
outlining needs to skip over lambdas that are args to idx. This PR adds
a small repro highlighting the issue.

Changed files:
filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/error.idx-outlining.expect.md
patch: @@ -0,0 +1,27 @@
+
+## Input
+
+```javascript
+import idx from 'idx';
+
+function Component(props) {
+  // the lambda should not be outlined
+  const groupName = idx(props, _ => _.group.label);
+  return <div>{groupName}</div>;
+}
+
+export const FIXTURE_ENTRYPOINT = {
+  fn: Component,
+  params: [{}],
+};
+
+```
+
+
+## Error
+
+```
+The second argument supplied to `idx` must be an arrow function. (This is an error on an internal node. Probably an internal error.)
+```
+          
+      
\ No newline at end of file

filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/error.idx-outlining.js
patch: @@ -0,0 +1,12 @@
+import idx from 'idx';
+
+function Component(props) {
+  // the lambda should not be outlined
+  const groupName = idx(props, _ => _.group.label);
+  return <div>{groupName}</div>;
+}
+
+export const FIXTURE_ENTRYPOINT = {
+  fn: Component,
+  params: [{}],
+};
Repository: ollama/ollama
Pull request: Add Testcontainers into Libraries section

Testcontainers provides a module for Ollama.


Changed files:
filename: README.md
patch: @@ -312,7 +312,6 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [LangChainGo](https://github.com/tmc/langchaingo/) with [example](https://github.com/tmc/langchaingo/tree/main/examples/ollama-completion-example)
 - [LangChain4j](https://github.com/langchain4j/langchain4j) with [example](https://github.com/langchain4j/langchain4j-examples/tree/main/ollama-examples/src/main/java)
 - [LlamaIndex](https://gpt-index.readthedocs.io/en/stable/examples/llm/ollama.html)
-- [LangChain4j](https://github.com/langchain4j/langchain4j/tree/main/langchain4j-ollama)
 - [LiteLLM](https://github.com/BerriAI/litellm)
 - [OllamaSharp for .NET](https://github.com/awaescher/OllamaSharp)
 - [Ollama for Ruby](https://github.com/gbaptista/ollama-ai)
@@ -329,6 +328,7 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [Ollama for R - rollama](https://github.com/JBGruber/rollama)
 - [Ollama-ex for Elixir](https://github.com/lebrunel/ollama-ex)
 - [Ollama Connector for SAP ABAP](https://github.com/b-tocs/abap_btocs_ollama)
+- [Testcontainers](https://testcontainers.com/modules/ollama/)
 
 ### Mobile
Repository: nodejs/node
Pull request: tools: avoid parsing test files twice

I've noticed that we're currently parsing the test files twice:

1. In the Python test runner
2. In `test/common/index.js`

By appending `NODE_SKIP_FLAG_CHECK` directly from the test runner, we can avoid this double check.
I'm opening this PR to evaluate whether the change provides any runtime benefits.

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
4. Update documentation if relevant.
5. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: tools/test.py
patch: @@ -788,6 +788,11 @@ def Execute(args, context, timeout=None, env=None, disable_core_files=False,
   for key, value in env.items():
     env_copy[key] = value
 
+  # We append NODE_SKIP_FLAG_CHECK (ref: test/common/index.js)
+  # to avoid parsing the test files twice when looking for
+  # flags or environment variables defined via // Flags: and // Env:
+  env_copy["NODE_SKIP_FLAG_CHECK"] = "true"
+
   preexec_fn = None
 
   def disableCoreFiles():