helloadhavan/github_issues huggingface dataset:
Repository: denoland/deno
Pull request: feat: ext/webgpu - add compatibility with {Free,Open}BSD

Both ports for the BSDs include patches to the same effect.

<!--
Before submitting a PR, please read https://docs.deno.com/runtime/manual/references/contributing

1. Give the PR a descriptive title.

  Examples of good title:
    - fix(std/http): Fix race condition in server
    - docs(console): Update docstrings
    - feat(doc): Handle nested reexports

  Examples of bad title:
    - fix #7123
    - update docs
    - fix bugs

2. Ensure there is a related issue and it is referenced in the PR text.
3. Ensure there are tests that cover the changes.
4. Ensure `cargo test` passes.
5. Ensure `./tools/format.js` passes without changing files.
6. Ensure `./tools/lint.js` passes.
7. Open as a draft PR if your work is still in progress. The CI won't run
   all steps, but you can add '[ci]' to a commit message to force it to.
8. If you would like to run the benchmarks on the CI, add the 'ci-bench' label.
-->


Changed files:
filename: ext/webgpu/byow.rs
patch: @@ -6,7 +6,12 @@ use deno_core::op2;
 use deno_core::OpState;
 use deno_core::ResourceId;
 use std::ffi::c_void;
-#[cfg(any(target_os = "linux", target_os = "macos"))]
+#[cfg(any(
+  target_os = "linux",
+  target_os = "macos",
+  target_os = "freebsd",
+  target_os = "openbsd"
+))]
 use std::ptr::NonNull;
 
 use crate::surface::WebGpuSurface;
@@ -103,7 +108,7 @@ fn raw_window(
   Ok((win_handle, display_handle))
 }
 
-#[cfg(target_os = "linux")]
+#[cfg(any(target_os = "linux", target_os = "freebsd", target_os = "openbsd"))]
 fn raw_window(
   system: &str,
   window: *const c_void,
@@ -136,7 +141,7 @@ fn raw_window(
       ),
     );
   } else {
-    return Err(type_error("Invalid system on Linux"));
+    return Err(type_error("Invalid system on Linux/BSD"));
   }
 
   Ok((win_handle, display_handle))
Repository: vercel/next.js
Pull request: [dev-overlay] remove text wrap for terminal

| Before | After |
|--------|--------|
| ![CleanShot 2025-03-10 at 18 51 16@2x](https://github.com/user-attachments/assets/2d39fcb5-072d-4ee0-a5bd-1df5e9c4b222) | ![CleanShot 2025-03-10 at 18 51 35@2x](https://github.com/user-attachments/assets/f0c21a0b-cca6-4606-b7a6-3ed8acc6e6c7) | 

Closes NDX-950

Changed files:
filename: packages/next/src/client/components/react-dev-overlay/ui/components/code-frame/code-frame.tsx
patch: @@ -136,10 +136,6 @@ export const CODE_FRAME_STYLES = `
     margin: auto 6px;
   }
 
-  .code-frame-pre {
-    white-space: pre-wrap;
-  }
-
   .code-frame-header {
     width: 100%;
     transition: background 100ms ease-out;
Repository: nodejs/node
Pull request: benchmark: add `test_runner/mock-fn`

Ref: #55723

This PR benchmarks the function-mocking of the test runner


Changed files:
filename: benchmark/test_runner/mock-fn.js
patch: @@ -0,0 +1,48 @@
+'use strict';
+
+const common = require('../common');
+const assert = require('node:assert');
+const { test } = require('node:test');
+
+const bench = common.createBenchmark(main, {
+  n: [1e6],
+  mode: ['define', 'execute'],
+}, {
+  // We don't want to test the reporter here
+  flags: ['--test-reporter=./benchmark/fixtures/empty-test-reporter.js'],
+});
+
+const noop = () => {};
+
+function benchmarkDefine(n) {
+  let noDead;
+  test((t) => {
+    bench.start();
+    for (let i = 0; i < n; i++) {
+      noDead = t.mock.fn(noop);
+    }
+    bench.end(n);
+    assert.ok(noDead);
+  });
+}
+
+function benchmarkExecute(n) {
+  let noDead;
+  test((t) => {
+    const mocked = t.mock.fn(noop);
+    bench.start();
+    for (let i = 0; i < n; i++) {
+      noDead = mocked();
+    }
+    bench.end(n);
+    assert.strictEqual(noDead, noop());
+  });
+}
+
+function main({ n, mode }) {
+  if (mode === 'define') {
+    benchmarkDefine(n);
+  } else if (mode === 'execute') {
+    benchmarkExecute(n);
+  }
+}
Repository: ggml-org/llama.cpp
Pull request: sync : ggml


Changed files:
filename: scripts/sync-ggml.last
patch: @@ -1 +1 @@
-c7dfe3d174f98b14801f9ed12f129179d3e7b638
+fc21aba88324312d66f62af6a1b0683fcb2ce3b5
Repository: denoland/deno
Pull request: fix(ext/node): return cancelled flag in get_response_body_chunk op

The flag lets us exit from read loop without throwing an error when
the stream is cancelled.

This fixes gRPC cancellation example.

Changed files:
filename: ext/node/ops/http2.rs
patch: @@ -24,6 +24,7 @@ use deno_core::ResourceId;
 use deno_net::raw::take_network_stream_resource;
 use deno_net::raw::NetworkStream;
 use h2;
+use h2::Reason;
 use h2::RecvStream;
 use http_v02;
 use http_v02::request::Parts;
@@ -496,17 +497,27 @@ fn poll_data_or_trailers(
 pub async fn op_http2_client_get_response_body_chunk(
   state: Rc<RefCell<OpState>>,
   #[smi] body_rid: ResourceId,
-) -> Result<(Option<Vec<u8>>, bool), AnyError> {
+) -> Result<(Option<Vec<u8>>, bool, bool), AnyError> {
   let resource = state
     .borrow()
     .resource_table
     .get::<Http2ClientResponseBody>(body_rid)?;
   let mut body = RcRef::map(&resource, |r| &r.body).borrow_mut().await;
 
   loop {
-    match poll_fn(|cx| poll_data_or_trailers(cx, &mut body)).await? {
+    let result = poll_fn(|cx| poll_data_or_trailers(cx, &mut body)).await;
+    if let Err(err) = result {
+      let reason = err.reason();
+      if let Some(reason) = reason {
+        if reason == Reason::CANCEL {
+          return Ok((None, false, true));
+        }
+      }
+      return Err(err.into());
+    }
+    match result.unwrap() {
       DataOrTrailers::Data(data) => {
-        return Ok((Some(data.to_vec()), false));
+        return Ok((Some(data.to_vec()), false, false));
       }
       DataOrTrailers::Trailers(trailers) => {
         if let Some(trailers_tx) = RcRef::map(&resource, |r| &r.trailers_tx)
@@ -524,7 +535,7 @@ pub async fn op_http2_client_get_response_body_chunk(
           .borrow_mut()
           .await
           .take();
-        return Ok((None, true));
+        return Ok((None, true, false));
       }
     };
   }

filename: ext/node/polyfills/http2.ts
patch: @@ -1007,9 +1007,14 @@ export class ClientHttp2Stream extends Duplex {
     debugHttp2(">>> read");
 
     (async () => {
-      const [chunk, finished] = await op_http2_client_get_response_body_chunk(
-        this[kDenoResponse].bodyRid,
-      );
+      const [chunk, finished, cancelled] =
+        await op_http2_client_get_response_body_chunk(
+          this[kDenoResponse].bodyRid,
+        );
+
+      if (cancelled) {
+        return;
+      }
 
       debugHttp2(">>> chunk", chunk, finished, this[kDenoResponse].bodyRid);
       if (chunk === null) {
Repository: facebook/react
Pull request: Enable enableSuspenseLayoutEffectSemantics flag statically for Facebook

This flag has been rolled out on Facebook at 10% public for ~3 weeks with no stat sig negative impact so let's enable it statically for Facebook builds.

Changed files:
filename: packages/shared/forks/ReactFeatureFlags.www-dynamic.js
patch: @@ -18,7 +18,6 @@ export const disableInputAttributeSyncing = __VARIANT__;
 export const enableFilterEmptyStringAttributesDOM = __VARIANT__;
 export const enableLegacyFBSupport = __VARIANT__;
 export const skipUnmountedBoundaries = __VARIANT__;
-export const enableSuspenseLayoutEffectSemantics = __VARIANT__;
 
 // Enable this flag to help with concurrent mode debugging.
 // It logs information to the console about React scheduling, rendering, and commit phases.

filename: packages/shared/forks/ReactFeatureFlags.www.js
patch: @@ -26,7 +26,6 @@ export const {
   enableDebugTracing,
   skipUnmountedBoundaries,
   createRootStrictEffectsByDefault,
-  enableSuspenseLayoutEffectSemantics,
   enableUseRefAccessWarning,
   disableNativeComponentFrames,
   disableSchedulerTimeoutInWorkLoop,
@@ -47,6 +46,8 @@ export const enableProfilerNestedUpdateScheduledHook =
   __PROFILE__ && dynamicFeatureFlags.enableProfilerNestedUpdateScheduledHook;
 export const enableUpdaterTracking = __PROFILE__;
 
+export const enableSuspenseLayoutEffectSemantics = true;
+
 // Logs additional User Timing API marks for use with an experimental profiling tool.
 export const enableSchedulingProfiler =
   __PROFILE__ && dynamicFeatureFlags.enableSchedulingProfiler;
Repository: huggingface/transformers
Pull request: Add ruff target-version

Target ruff to py3.9

Changed files:
filename: pyproject.toml
patch: @@ -14,6 +14,7 @@ exclude_lines = [
 ]
 
 [tool.ruff]
+target-version = "py39"
 line-length = 119
 
 [tool.ruff.lint]
@@ -53,4 +54,4 @@ markers = [
     "generate: marks tests that use the GenerationTesterMixin"
 ]
 log_cli = 1
-log_cli_level = "WARNING"
\ No newline at end of file
+log_cli_level = "WARNING"
Repository: vercel/next.js
Pull request: [dev-overlay] Fix unactionable `useLayoutEffect` warning if React 18 is used

Fixes the old "useLayoutEffect does nothing on the Server" in React 18. Noticed it while working on https://github.com/vercel/next.js/pull/77326

Changed files:
filename: packages/next/src/client/components/react-dev-overlay/ui/components/shadow-portal.tsx
patch: @@ -7,7 +7,11 @@ export function ShadowPortal({ children }: { children: React.ReactNode }) {
   let shadowNode = React.useRef<ShadowRoot | null>(null)
   let [, forceUpdate] = React.useState<{} | undefined>()
 
-  React.useLayoutEffect(() => {
+  // Don't use useLayoutEffect here, as it will cause warnings during SSR in React 18.
+  // Don't use useSyncExternalStore as an SSR gate unless you verified it doesn't
+  // downgrade a Transition of the initial root render to a sync render or
+  // we can assure the root render is not a Transition.
+  React.useEffect(() => {
     const ownerDocument = document
     portalNode.current = ownerDocument.createElement('nextjs-portal')
     // load default color preference from localstorage
Repository: nodejs/node
Pull request: doc: run license-builder

License is likely out of date. This is an automatically generated PR by the `license-builder.yml` GitHub Action, which runs `license-builder.sh` and submits a new PR or updates an existing PR.

Changed files:
filename: LICENSE
patch: @@ -797,6 +797,34 @@ The externally maintained libraries used by Node.js are:
 
     ----------------------------------------------------------------------
 
+    JSON parsing library (nlohmann/json)
+
+    File: vendor/json/upstream/single_include/nlohmann/json.hpp (only for ICU4C)
+
+    MIT License
+
+    Copyright (c) 2013-2022 Niels Lohmann
+
+    Permission is hereby granted, free of charge, to any person obtaining a copy
+    of this software and associated documentation files (the "Software"), to deal
+    in the Software without restriction, including without limitation the rights
+    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+    copies of the Software, and to permit persons to whom the Software is
+    furnished to do so, subject to the following conditions:
+
+    The above copyright notice and this permission notice shall be included in all
+    copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+    SOFTWARE.
+
+    ----------------------------------------------------------------------
+
     File: aclocal.m4 (only for ICU4C)
     Section: pkg.m4 - Macros to locate and utilise pkg-config.
 
@@ -834,7 +862,7 @@ The externally maintained libraries used by Node.js are:
 
     This file is free software; you can redistribute it and/or modify it
     under the terms of the GNU General Public License as published by
-    the Free Software Foundation; either version 3 of the License, or
+    the Free Software Foundation, either version 3 of the License, or
     (at your option) any later version.
 
     This program is distributed in the hope that it will be useful, but
Repository: denoland/deno
Pull request: fix(ext/node): add stubs for perf_hooks.PerformaceObserver

Fixes https://github.com/denoland/deno/issues/23943

Changed files:
filename: ext/node/polyfills/perf_hooks.ts
patch: @@ -9,8 +9,15 @@ import {
   PerformanceEntry,
 } from "ext:deno_web/15_performance.js";
 
-// FIXME(bartlomieju)
-const PerformanceObserver = undefined;
+class PerformanceObserver {
+  observe() {
+    notImplemented("PerformanceObserver.observe");
+  }
+  disconnect() {
+    notImplemented("PerformanceObserver.disconnect");
+  }
+}
+
 const constants = {};
 
 const performance:

filename: tests/unit_node/perf_hooks_test.ts
patch: @@ -1,6 +1,6 @@
 // Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
 import * as perfHooks from "node:perf_hooks";
-import { performance } from "node:perf_hooks";
+import { performance, PerformanceObserver } from "node:perf_hooks";
 import { assertEquals, assertThrows } from "@std/assert/mod.ts";
 
 Deno.test({
@@ -44,9 +44,10 @@ Deno.test({
 });
 
 Deno.test({
-  name: "[perf_hooks] PerformanceEntry",
+  name: "[perf_hooks] PerformanceEntry & PerformanceObserver",
   fn() {
     assertEquals<unknown>(perfHooks.PerformanceEntry, PerformanceEntry);
+    assertEquals<unknown>(perfHooks.PerformanceObserver, PerformanceObserver);
   },
 });
Repository: electron/electron
Pull request: fix: redirect webview navigation methods

Backport of #42981

See that PR for details.


Notes: none


Changed files:
filename: lib/browser/guest-view-manager.ts
patch: @@ -2,7 +2,7 @@ import { webContents } from 'electron/main';
 import { ipcMainInternal } from '@electron/internal/browser/ipc-main-internal';
 import * as ipcMainUtils from '@electron/internal/browser/ipc-main-internal-utils';
 import { parseWebViewWebPreferences } from '@electron/internal/browser/parse-features-string';
-import { syncMethods, asyncMethods, properties } from '@electron/internal/common/web-view-methods';
+import { syncMethods, asyncMethods, properties, navigationHistorySyncMethods } from '@electron/internal/common/web-view-methods';
 import { webViewEvents } from '@electron/internal/browser/web-view-events';
 import { IPC_MESSAGES } from '@electron/internal/common/ipc-messages';
 
@@ -311,6 +311,14 @@ handleMessageSync(IPC_MESSAGES.GUEST_VIEW_MANAGER_CALL, function (event, guestIn
   if (!syncMethods.has(method)) {
     throw new Error(`Invalid method: ${method}`);
   }
+  // Redirect history methods to updated navigationHistory property on webContents. See issue #42879.
+  if (navigationHistorySyncMethods.has(method)) {
+    let navigationMethod = method;
+    if (method === 'clearHistory') {
+      navigationMethod = 'clear';
+    }
+    return (guest as any).navigationHistory[navigationMethod](...args);
+  }
 
   return (guest as any)[method](...args);
 });

filename: lib/common/web-view-methods.ts
patch: @@ -1,3 +1,14 @@
+export const navigationHistorySyncMethods = new Set([
+  'canGoBack',
+  'canGoForward',
+  'canGoToOffset',
+  'clearHistory',
+  'goBack',
+  'goForward',
+  'goToIndex',
+  'goToOffset'
+]);
+
 // Public-facing API methods.
 export const syncMethods = new Set([
   'getURL',
@@ -8,14 +19,6 @@ export const syncMethods = new Set([
   'stop',
   'reload',
   'reloadIgnoringCache',
-  'canGoBack',
-  'canGoForward',
-  'canGoToOffset',
-  'clearHistory',
-  'goBack',
-  'goForward',
-  'goToIndex',
-  'goToOffset',
   'isCrashed',
   'setUserAgent',
   'getUserAgent',
@@ -51,7 +54,8 @@ export const syncMethods = new Set([
   'getZoomFactor',
   'getZoomLevel',
   'setZoomFactor',
-  'setZoomLevel'
+  'setZoomLevel',
+  ...navigationHistorySyncMethods
 ]);
 
 export const properties = new Set([
Repository: facebook/react
Pull request: Fix typo in parseHookNames.js

Fix typo:

representated -> represented


Changed files:
filename: packages/react-devtools-extensions/src/parseHookNames/parseHookNames.js
patch: @@ -391,7 +391,7 @@ function findHookNames(
       const position = sourceConsumer.originalPositionFor({
         line: lineNumber,
 
-        // Column numbers are representated differently between tools/engines.
+        // Column numbers are represented differently between tools/engines.
         // Error.prototype.stack columns are 1-based (like most IDEs) but ASTs are 0-based.
         // For more info see https://github.com/facebook/react/issues/21792#issuecomment-873171991
         column: columnNumber - 1,
@@ -490,7 +490,7 @@ async function parseSourceAST(
       const {source} = sourceConsumer.originalPositionFor({
         line: lineNumber,
 
-        // Column numbers are representated differently between tools/engines.
+        // Column numbers are represented differently between tools/engines.
         // Error.prototype.stack columns are 1-based (like most IDEs) but ASTs are 0-based.
         // For more info see https://github.com/facebook/react/issues/21792#issuecomment-873171991
         column: columnNumber - 1,
Repository: nodejs/node
Pull request: build: implement node_use_amaro flag in GN build

Implement the `node_use_amaro` flag in GN build, which was added to GYP build in https://github.com/nodejs/node/pull/54136.

Refs: https://github.com/nodejs/node/pull/54136

Changed files:
filename: node.gni
patch: @@ -59,6 +59,9 @@ declare_args() {
   #   1. cross-os compilation is not supported.
   #   2. node_mksnapshot crashes when cross-compiling for x64 from arm64.
   node_use_node_snapshot = (host_os == target_os) && !(host_cpu == "arm64" && target_cpu == "x64")
+
+  # Build with Amaro (TypeScript utils).
+  node_use_amaro = true
 }
 
 assert(!node_enable_inspector || node_use_openssl,

filename: tools/generate_config_gypi.py
patch: @@ -61,6 +61,7 @@ def translate_config(out_dir, config, v8_config):
           eval(config['node_builtin_shareable_builtins']),
       'node_module_version': int(config['node_module_version']),
       'node_use_openssl': config['node_use_openssl'],
+      'node_use_amaro': config['node_use_amaro'],
       'node_use_node_code_cache': config['node_use_node_code_cache'],
       'node_use_node_snapshot': config['node_use_node_snapshot'],
       'v8_enable_inspector':  # this is actually a node misnomer

filename: unofficial.gni
patch: @@ -22,6 +22,11 @@ template("node_gn_build") {
     } else {
       defines += [ "HAVE_OPENSSL=0" ]
     }
+    if (node_use_amaro) {
+      defines += [ "HAVE_AMARO=1" ]
+    } else {
+      defines += [ "HAVE_AMARO=0" ]
+    }
     if (node_use_v8_platform) {
       defines += [ "NODE_USE_V8_PLATFORM=1" ]
     } else {
Repository: ggml-org/llama.cpp
Pull request: convert : fix squeeze for ssm_conv tensors

fix https://github.com/ggml-org/llama.cpp/issues/12572

Broken in [#10784](https://github.com/ggml-org/llama.cpp/pull/10784/files#diff-ec77d8003b92ff283179456d36b8b56abf635e7b1232e70daf16676e8920ccf1R299-R301).

Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -3803,8 +3803,6 @@ def set_gguf_parameters(self):
     _tok_embd = None
 
     def modify_tensors(self, data_torch: Tensor, name: str, bid: int | None) -> Iterable[tuple[str, Tensor]]:
-        del bid  # unused
-
         output_name = self.format_tensor_name(gguf.MODEL_TENSOR.OUTPUT)
         tok_embd_name = self.format_tensor_name(gguf.MODEL_TENSOR.TOKEN_EMBD)
 
@@ -3814,6 +3812,10 @@ def modify_tensors(self, data_torch: Tensor, name: str, bid: int | None) -> Iter
             logger.debug("A_log --> A ==> " + new_name)
             data_torch = -torch.exp(data_torch)
 
+        # [4 1 8192 1] -> [4 8192 1 1]
+        if self.match_model_tensor_name(new_name, gguf.MODEL_TENSOR.SSM_CONV1D, bid):
+            data_torch = data_torch.squeeze()
+
         # assuming token_embd.weight is seen before output.weight
         if self._tok_embd is not None and new_name == output_name:
             if torch.equal(self._tok_embd, data_torch):
Repository: denoland/deno
Pull request: perf(startup): use WAL journal for sqlite databases in DENO_DIR

While investigating poor cold start performance on my GCP VM (32 cores, 130GB SSD), I found that writing to the various sqlite databases in DENO_DIR was quite slow. The slowness seems to primarily be caused by excessive latency from a number of `fsync()` calls.

The performance difference is best demonstrated by deleting the sqlite databases from DENO_DIR while leaving the downloaded sources in place.

The benchmark (see notes below):

```
piscisaureus@bert-us:~/erofs/source$ export DENO_DIR=./.deno
piscisaureus@bert-us:~/erofs/source$ hyperfine --warmup 3   \
  --prepare "rm -rf .deno/*_v1*"                            \
  "deno run -A --cached-only demo.ts"                       \
  "eatmydata deno run -A --cached-only demo.ts"             \
  "~/deno/target/release/deno run -A --cached-only demo.ts"
Benchmark 1: deno run -A --cached-only demo.ts
  Time (mean ± σ):      1.174 s ±  0.037 s    [User: 0.153 s, System: 0.184 s]
  Range (min … max):    1.104 s …  1.212 s    10 runs
 
Benchmark 2: eatmydata deno run -A --cached-only demo.ts
  Time (mean ± σ):     265.5 ms ±   3.6 ms    [User: 138.5 ms, System: 135.1 ms]
  Range (min … max):   260.6 ms … 271.2 ms    11 runs
 
Benchmark 3: ~/deno/target/release/deno run -A --cached-only demo.ts
  Time (mean ± σ):     226.2 ms ±   9.2 ms    [User: 136.7 ms, System: 93.3 ms]
  Range (min … max):   218.8 ms … 247.1 ms    13 runs
 
Summary
  ~/deno/target/release/deno run -A --cached-only demo.ts ran
    1.17 ± 0.05 times faster than eatmydata deno run -A --cached-only demo.ts
    5.19 ± 0.27 times faster than deno run -A --cached-only demo.ts
```

Notes:
* Benchmark 1: unmodified Deno 1.43.6
* Benchmark 2: unmodified Deno 1.43.6 wrapped with `eatmydata` (which is a tool to neuter `fsync()` calls)
* Benchmark 3: this PR applied on top of Deno 1.43.6


The script that got benchmarked:

```typescript
// demo.ts
import * as express from "npm:express@4.16.3";
import * as postgres from "https://deno.land/x/postgres/mod.ts";

let _dummy = [express, postgres]; // Force use of imports.
console.log("hello world");
```


Changed files:
filename: cli/cache/cache_db.rs
patch: @@ -42,7 +42,7 @@ impl CacheDBConfiguration {
   fn create_combined_sql(&self) -> String {
     format!(
       "
-      PRAGMA journal_mode=TRUNCATE;
+      PRAGMA journal_mode=WAL;
       PRAGMA synchronous=NORMAL;
       PRAGMA temp_store=memory;
       PRAGMA page_size=4096;
Repository: huggingface/transformers
Pull request: update bot comment again

# What does this PR do?

update bot comment again, as a new CircleCI pipeline with a new trigger (by ready for review event) is added.

Changed files:
filename: .github/workflows/change_pr_to_draft.yml
patch: @@ -22,4 +22,4 @@ jobs:
         run: |
           echo $PR_NUMBER
           gh pr ready $PR_NUMBER --repo $REPO --undo
-          gh pr comment $PR_NUMBER --repo $REPO --body "Hi 👋, thank you for opening this pull request! The pull request is converted to draft by default. The CI will be paused while the PR is in draft mode. When it is ready for review, please click the \`Ready for review\` button (at the bottom of the PR page). This will assign reviewers."
+          gh pr comment $PR_NUMBER --repo $REPO --body "Hi 👋, thank you for opening this pull request! The pull request is converted to draft by default. The CI will be paused while the PR is in draft mode. When it is ready for review, please click the \`Ready for review\` button (at the bottom of the PR page). This will assign reviewers and trigger CI."