helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: [test] Add test for dev rendering indicator

Automated test for https://github.com/vercel/next.js/pull/74833

Changed files:
filename: test/development/dev-indicator/app/app/rendering/a/page.js
patch: @@ -0,0 +1,16 @@
+import Link from 'next/link'
+import { connection } from 'next/server'
+import { setTimeout } from 'timers/promises'
+
+export default async function Page() {
+  await connection()
+  await setTimeout(100)
+
+  return (
+    <>
+      <Link href="/app/rendering/b" id="to-b">
+        Go to b
+      </Link>
+    </>
+  )
+}

filename: test/development/dev-indicator/app/app/rendering/b/page.js
patch: @@ -0,0 +1,15 @@
+import Link from 'next/link'
+import { connection } from 'next/server'
+import { setTimeout } from 'timers/promises'
+
+export default async function Page() {
+  await connection()
+  await setTimeout(100)
+  return (
+    <>
+      <Link href="/app/rendering/a" id="to-a">
+        Go to a
+      </Link>
+    </>
+  )
+}

filename: test/development/dev-indicator/app/layout.js
patch: @@ -0,0 +1,7 @@
+export default function RootLayout({ children }) {
+  return (
+    <html lang="en">
+      <body>{children}</body>
+    </html>
+  )
+}

filename: test/development/dev-indicator/dev-rendering-indicator.test.ts
patch: @@ -0,0 +1,44 @@
+/* eslint-env jest */
+
+import { nextTestSetup } from 'e2e-utils'
+import { retry } from 'next-test-utils'
+
+const installCheckVisible = (browser) => {
+  return browser.eval(`(function() {
+      window.checkInterval = setInterval(function() {
+      const root = document.querySelector('nextjs-portal').shadowRoot;
+      const indicator = root.querySelector('[data-next-mark]')
+      window.showedBuilder = window.showedBuilder || (
+        indicator.getAttribute('data-next-mark-loading') === 'true'
+      )
+      if (window.showedBuilder) clearInterval(window.checkInterval)
+    }, 5)
+  })()`)
+}
+
+describe('Dev Rendering Indicator', () => {
+  const { next } = nextTestSetup({
+    files: __dirname,
+  })
+
+  it('Shows build indicator when page is built from modifying', async () => {
+    // Ensure both pages are built first so that we don't confuse it with build indicator
+    await Promise.all([
+      next.fetch('/app/rendering/a'),
+      next.fetch('/app/rendering/b'),
+    ])
+    const browser = await next.browser('/app/rendering/a')
+    await installCheckVisible(browser)
+    await browser.eval('window.showedBuilder = false')
+
+    await browser.elementByCss('[href="/app/rendering/b"]').click()
+    await retry(async () => {
+      await browser.elementByCss('[href="/app/rendering/a"]')
+    })
+
+    const showedRenderingIndicator = await browser.eval('window.showedBuilder')
+    expect({ showedRenderingIndicator }).toEqual({
+      showedRenderingIndicator: true,
+    })
+  })
+})
Repository: denoland/deno
Pull request: test(ext/node): enable parallel/test-fs-promises-file-handle-stat.js

This PR enables node compat test case `parallel/test-fs-promises-file-handle-stat.js`. (FileHandle#stat was implemented in https://github.com/denoland/deno/pull/26719 )

Changed files:
filename: tests/node_compat/config.jsonc
patch: @@ -338,6 +338,7 @@
       "test-fs-open-numeric-flags.js",
       "test-fs-open.js",
       "test-fs-opendir.js",
+      "test-fs-promises-file-handle-stat.js",
       "test-fs-promises-writefile-with-fd.js",
       "test-fs-read-stream-autoClose.js",
       "test-fs-read-stream-concurrent-reads.js",

filename: tests/node_compat/runner/TODO.md
patch: @@ -822,7 +822,6 @@ NOTE: This file should not be manually edited. Please edit `tests/node_compat/co
 - [parallel/test-fs-promises-file-handle-read-worker.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-read-worker.js)
 - [parallel/test-fs-promises-file-handle-read.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-read.js)
 - [parallel/test-fs-promises-file-handle-readFile.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-readFile.js)
-- [parallel/test-fs-promises-file-handle-stat.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-stat.js)
 - [parallel/test-fs-promises-file-handle-stream.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-stream.js)
 - [parallel/test-fs-promises-file-handle-sync.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-sync.js)
 - [parallel/test-fs-promises-file-handle-truncate.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-fs-promises-file-handle-truncate.js)

filename: tests/node_compat/test/parallel/test-fs-promises-file-handle-stat.js
patch: @@ -0,0 +1,30 @@
+// deno-fmt-ignore-file
+// deno-lint-ignore-file
+
+// Copyright Joyent and Node contributors. All rights reserved. MIT license.
+// Taken from Node 20.11.1
+// This file is automatically generated by `tests/node_compat/runner/setup.ts`. Do not modify this file manually.
+
+'use strict';
+
+const common = require('../common');
+
+// The following tests validate base functionality for the fs.promises
+// FileHandle.stat method.
+
+const { open } = require('fs').promises;
+const tmpdir = require('../common/tmpdir');
+const assert = require('assert');
+
+tmpdir.refresh();
+
+async function validateStat() {
+  const filePath = tmpdir.resolve('tmp-read-file.txt');
+  const fileHandle = await open(filePath, 'w+');
+  const stats = await fileHandle.stat();
+  assert.ok(stats.mtime instanceof Date);
+  await fileHandle.close();
+}
+
+validateStat()
+  .then(common.mustCall());
Repository: facebook/react
Pull request: Rename "dom" fork to "dom-node" and "bun" fork to "dom-bun"

The "dom" configuration is actually the node specific configuration. It just happened to be that this was the mainline variant before so it was implied but with so many variants, this is less obvious now.

The "bun" configuration is specifically for "bun". There's no "native" renderer for "bun" yet.


Changed files:
filename: scripts/shared/inlinedHostConfigs.js
patch: @@ -8,7 +8,7 @@
 
 module.exports = [
   {
-    shortName: 'dom',
+    shortName: 'dom-node',
     entryPoints: [
       'react-dom',
       'react-dom/unstable_testing',
@@ -46,7 +46,7 @@ module.exports = [
     isServerSupported: true,
   },
   {
-    shortName: 'bun',
+    shortName: 'dom-bun',
     entryPoints: ['react-dom', 'react-dom/src/server/ReactDOMFizzServerBun.js'],
     paths: [
       'react-dom',
Repository: denoland/deno
Pull request: fix(tools/doc): HTML resolve main entrypoint from config file 

Fixes #26901

Changed files:
filename: cli/tools/doc.rs
patch: @@ -209,10 +209,14 @@ pub async fn doc(
       Default::default()
     };
 
+    let mut main_entrypoint = None;
+
     let rewrite_map =
       if let Some(config_file) = cli_options.start_dir.maybe_deno_json() {
         let config = config_file.to_exports_config()?;
 
+        main_entrypoint = config.get_resolved(".").ok().flatten();
+
         let rewrite_map = config
           .clone()
           .into_map()
@@ -240,6 +244,7 @@ pub async fn doc(
       html_options,
       deno_ns,
       rewrite_map,
+      main_entrypoint,
     )
   } else {
     let modules_len = doc_nodes_by_url.len();
@@ -383,6 +388,7 @@ fn generate_docs_directory(
   html_options: &DocHtmlFlag,
   deno_ns: std::collections::HashMap<Vec<String>, Option<Rc<ShortPath>>>,
   rewrite_map: Option<IndexMap<ModuleSpecifier, String>>,
+  main_entrypoint: Option<ModuleSpecifier>,
 ) -> Result<(), AnyError> {
   let cwd = std::env::current_dir().context("Failed to get CWD")?;
   let output_dir_resolved = cwd.join(&html_options.output);
@@ -415,7 +421,7 @@ fn generate_docs_directory(
 
   let options = deno_doc::html::GenerateOptions {
     package_name: html_options.name.clone(),
-    main_entrypoint: None,
+    main_entrypoint,
     rewrite_map,
     href_resolver: Rc::new(DocResolver {
       deno_ns,

filename: tests/specs/doc/html/__test__.jsonc
patch: @@ -1,12 +1,21 @@
 {
   "tempDir": true,
-  "args": [
-    "doc",
-    "--html",
-    "--name=MyLib",
-    "--output=temp_dir_path_here",
-    "referenced_private_types_fixed.ts"
-  ],
-  "output": "[WILDCARD]",
-  "exitCode": 0
+  "steps": [
+    {
+      "args": [
+        "doc",
+        "--html",
+        "--name=MyLib",
+        "a.ts",
+        "b.ts"
+      ],
+      "output": "Written 23 files to \"./docs/\"\n",
+      "exitCode": 0
+    },
+    {
+      "args": "run --allow-read check_file.ts",
+      "output": "",
+      "exitCode": 0
+    }
+  ]
 }

filename: tests/specs/doc/html/b.ts
patch: @@ -0,0 +1,11 @@
+/** Doc comment */
+export interface MyInterface2 {
+  /** Doc comment */
+  prop?: string;
+}
+
+/** Doc comment */
+export class MyClass2 {
+  /** Doc comment */
+  prop: MyInterface2 = {};
+}

filename: tests/specs/doc/html/check_file.ts
patch: @@ -0,0 +1,5 @@
+const content = Deno.readTextFileSync("./docs/index.html");
+
+if (content.includes("..")) {
+  throw new Error();
+}

filename: tests/specs/doc/html/deno.json
patch: @@ -0,0 +1,5 @@
+{
+  "exports": {
+    ".": "./a.ts"
+  }
+}
Repository: vercel/next.js
Pull request: [test] fix flaky test of basepath navigation

### What

The navigation test case in basepath is flaky. When the navigation happened, we need to wait for the url change otherwise the element assertion is easy to fail on CI.

x-ref: https://github.com/vercel/next.js/actions/runs/15473866317/job/43564737619?pr=80212

```
● app dir - basepath › should successfully hard navigate from pages -> app

    page.waitForSelector: Timeout 10000ms exceeded.
    Call log:
      - waiting for locator('#page-2')

      454 |   waitForElementByCss(selector: string, timeout = 10_000) {
      455 |     return this.startChain(async () => {
    > 456 |       const el = await page.waitForSelector(selector, {
          |                             ^
      457 |         timeout,
      458 |         state: 'attached',
      459 |       })
```

Changed files:
filename: test/e2e/app-dir/app-basepath/index.test.ts
patch: @@ -12,7 +12,12 @@ describe('app dir - basepath', () => {
 
   it('should successfully hard navigate from pages -> app', async () => {
     const browser = await next.browser('/base/pages-path')
-    await browser.elementByCss('#to-another').click()
+    await browser.waitForElementByCss('#to-another').click()
+    // Wait for url to change, ensure the navigation is finished,
+    // then we can assert if the element is present.
+    await retry(async () => {
+      expect(await browser.url()).toBe(`${next.url}/base/another`)
+    }, 5_000)
     await browser.waitForElementByCss('#page-2')
   })
Repository: facebook/react
Pull request: publish-prereleases: Exit if CircleCI request fails

If the publish-prereleases command fails to access CircleCI, it will now exit with a message instead of hanging indefinitely.

Changed files:
filename: scripts/release/publish-using-ci-workflow.js
patch: @@ -99,6 +99,15 @@ async function main() {
     }
   );
 
+  if (!pipelineResponse.ok) {
+    console.error(
+      theme.error(
+        `Failed to access CircleCI. Responded with status: ${pipelineResponse.status}`
+      )
+    );
+    process.exit(1);
+  }
+
   const pipelineJSON = await pipelineResponse.json();
   const pipelineID = pipelineJSON.id;
Repository: huggingface/transformers
Pull request: Don't run `AriaForConditionalGenerationModelTest` on CircleCI

# What does this PR do?

Let's just simply get rid of this model tests on CircleCI. Cause the slowest job to finish in `13m 25s`.

![Screenshot 2025-06-05 165750](https://github.com/user-attachments/assets/21c7f1d2-a7bc-4f53-93ee-929748dfbb75)


We still have it on dailly CI


```bash
148.80s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_model_outputs_equivalence
120.40s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_model_outputs_equivalence
55.83s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_resize_tokens_embeddings
36.21s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_attention_outputs
28.76s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_resize_embeddings_untied
23.11s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_training
22.56s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_retain_grad_hidden_states_attentions
22.27s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_hidden_states_output
22.15s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_determinism
14.41s call     tests/models/aria/test_modeling_aria.py::AriaForConditionalGenerationModelTest::test_save_load
```

Changed files:
filename: tests/models/aria/test_modeling_aria.py
patch: @@ -171,6 +171,7 @@ def prepare_config_and_inputs_for_common(self):
         return config, inputs_dict
 
 
+@slow
 @require_torch
 class AriaForConditionalGenerationModelTest(ModelTesterMixin, GenerationTesterMixin, unittest.TestCase):
     """
Repository: ggml-org/llama.cpp
Pull request: model card yaml tab->2xspace

Issue to fix:

```
yaml.scanner.ScannerError: while scanning for the next token
found character '\t' that cannot start any token
  in "<unicode string>", line 2, column 20:
    license: apache-2.0	
```

caused by Qwen/Qwen3-235B-A22B-Instruct-2507 model card had spaces

yaml should use 2 spaces instead of tab.

Tested locally

(Note: I've also sent a PR to fix the model card too)


Changed files:
filename: gguf-py/gguf/metadata.py
patch: @@ -144,6 +144,10 @@ def load_model_card(model_path: Optional[Path] = None) -> dict[str, Any]:
         # Quick hack to fix the Norway problem
         # https://hitchdev.com/strictyaml/why/implicit-typing-removed/
         yaml_content = yaml_content.replace("- no\n", "- \"no\"\n")
+        # yaml should use 2 spaces insted of tab
+        # this issue has came up with the Qwen/Qwen3-235B-A22B-Instruct-2507 model card
+        #    (I've also sent a pr tp fix the modelcard too)
+        yaml_content = yaml_content.replace("\t", "  ")
 
         if yaml_content:
             data = yaml.safe_load(yaml_content)
Repository: denoland/deno
Pull request: fix(task): handle carriage return in task description

Ref https://github.com/denoland/deno/pull/27069#discussion_r1857702814

Changed files:
filename: cli/tools/task.rs
patch: @@ -721,7 +721,7 @@ fn print_available_tasks(
     )?;
     if let Some(description) = &desc.task.description {
       let slash_slash = colors::italic_gray("//");
-      for line in description.split('\n') {
+      for line in description.lines() {
         writeln!(writer, "    {slash_slash} {}", colors::italic_gray(line))?;
       }
     }

filename: tests/specs/task/description/deno.json
patch: @@ -7,6 +7,10 @@
     "multiline_description": {
       "description": "This is a multiline\ndescription",
       "command": "echo 2"
+    },
+    "multiline_description_carriage_return": {
+      "description": "This is a multiline\r\ndescription\r\n",
+      "command": "echo 3"
     }
   }
 }

filename: tests/specs/task/description/main.out
patch: @@ -6,3 +6,7 @@ Available tasks:
     // This is a multiline
     // description
     echo 2
+- multiline_description_carriage_return
+    // This is a multiline
+    // description
+    echo 3
Repository: ggml-org/llama.cpp
Pull request: vulkan: fix rms_norm_mul to handle broadcasting dim0

For the new tests in #14800.


Changed files:
filename: ggml/src/ggml-vulkan/ggml-vulkan.cpp
patch: @@ -10248,7 +10248,7 @@ static bool ggml_vk_can_fuse(const struct ggml_cgraph * cgraph, int node_idx, st
         }
         // if rms_norm is the B operand, then we don't handle broadcast
         if (rms_norm == mul->src[1] &&
-            mul->src[0]->ne[1] != rms_norm->ne[1]) {
+            !ggml_are_same_shape(mul->src[0], rms_norm)) {
             return false;
         }
         // rms_norm shader assumes contiguous rows

filename: ggml/src/ggml-vulkan/vulkan-shaders/rms_norm.comp
patch: @@ -50,8 +50,14 @@ void main() {
     const FLOAT_TYPE scale = inversesqrt(mean + FLOAT_TYPE(p.param1));
 
     if (do_multiply) {
-        [[unroll]] for (uint col = tid; col < ncols; col += BLOCK_SIZE) {
-            data_d[d_offset + col] = D_TYPE(scale * FLOAT_TYPE(data_a[a_offset + col]) * FLOAT_TYPE(data_b[b_offset + col]));
+        if (ncols > p.ne10) {
+            [[unroll]] for (uint col = tid; col < ncols; col += BLOCK_SIZE) {
+                data_d[d_offset + col] = D_TYPE(scale * FLOAT_TYPE(data_a[a_offset + col]) * FLOAT_TYPE(data_b[b_offset + fastmod(col, p.ne10)]));
+            }
+        } else {
+            [[unroll]] for (uint col = tid; col < ncols; col += BLOCK_SIZE) {
+                data_d[d_offset + col] = D_TYPE(scale * FLOAT_TYPE(data_a[a_offset + col]) * FLOAT_TYPE(data_b[b_offset + col]));
+            }
         }
     } else {
         [[unroll]] for (uint col = tid; col < ncols; col += BLOCK_SIZE) {
Repository: facebook/react
Pull request: Fix async test in React reconciler

## Summary

- Remove unnecessary done() callback
- Jest will throw an error, if the same test function is passed a done() callback and returns a promise. https://jestjs.io/docs/asynchronous#callbacks

## How did you test this change?

ci green


Changed files:
filename: packages/react-reconciler/src/__tests__/ReactUpdaters-test.internal.js
patch: @@ -255,7 +255,7 @@ describe('updaters', () => {
     Scheduler.unstable_flushAll();
   });
 
-  it('should cover suspense pings', async done => {
+  it('should cover suspense pings', async () => {
     let data = null;
     let resolver = null;
     let promise = null;
@@ -312,8 +312,6 @@ describe('updaters', () => {
 
     // Verify no outstanding flushes
     Scheduler.unstable_flushAll();
-
-    done();
   });
 
   it('should cover error handling', async () => {
Repository: huggingface/transformers
Pull request: Skip `test_initialization` for `SwiftFormer`

# What does this PR do?

`SwiftFormer` has

```
class SwiftFormerEfficientAdditiveAttention(nn.Module):
    def __init__(self, config: SwiftFormerConfig, dim: int = 512):
        super().__init__()
        self.w_g = nn.Parameter(torch.randn(dim, 1))

```
and `self.w_g` is not handled by `_init_weights` and also not by any thing like `initializer_range`, so it just a normal distribution with mean 0 and std 1, and make `test_initialization` quite flaky (> 10% failure rate).

This PR just skip `w_g` in the (already overwritten) test.





Changed files:
filename: tests/models/swiftformer/test_modeling_swiftformer.py
patch: @@ -250,6 +250,8 @@ def _config_zero_init(config):
         for model_class in self.all_model_classes:
             model = model_class(config=configs_no_init)
             for name, param in model.named_parameters():
+                if name.endswith(".w_g"):
+                    continue
                 if param.requires_grad:
                     self.assertIn(
                         ((param.data.mean() * 1e9) / 1e9).round().item(),
Repository: denoland/deno
Pull request: fix(task): handle multiline descriptions properly

Closes https://github.com/denoland/deno/issues/27049

Changed files:
filename: cli/tools/task.rs
patch: @@ -721,11 +721,9 @@ fn print_available_tasks(
     )?;
     if let Some(description) = &desc.task.description {
       let slash_slash = colors::italic_gray("//");
-      writeln!(
-        writer,
-        "    {slash_slash} {}",
-        colors::italic_gray(description)
-      )?;
+      for line in description.split('\n') {
+        writeln!(writer, "    {slash_slash} {}", colors::italic_gray(line))?;
+      }
     }
     writeln!(writer, "    {}", desc.task.command)?;
     if !desc.task.dependencies.is_empty() {

filename: tests/specs/task/description/deno.json
patch: @@ -3,6 +3,10 @@
     "echo_emoji": {
       "description": "This is some task",
       "command": "echo 1"
+    },
+    "multiline_description": {
+      "description": "This is a multiline\ndescription",
+      "command": "echo 2"
     }
   }
 }

filename: tests/specs/task/description/main.out
patch: @@ -2,3 +2,7 @@ Available tasks:
 - echo_emoji
     // This is some task
     echo 1
+- multiline_description
+    // This is a multiline
+    // description
+    echo 2
Repository: facebook/react
Pull request: Remove unused dependency 'abort-controller'

## Summary
This PR removes the unused dependency 'abort-controller' from the project. it helps to keep the project clean and maintainable.

## How did you test this change?
ci green

Changed files:
filename: package.json
patch: @@ -36,7 +36,6 @@
     "@babel/preset-flow": "^7.10.4",
     "@babel/preset-react": "^7.10.4",
     "@babel/traverse": "^7.11.0",
-    "abort-controller": "^3.0.0",
     "art": "0.10.1",
     "babel-plugin-syntax-trailing-function-commas": "^6.5.0",
     "chalk": "^3.0.0",

filename: yarn.lock
patch: @@ -3274,13 +3274,6 @@ abab@^2.0.5:
   resolved "https://registry.yarnpkg.com/abab/-/abab-2.0.5.tgz#c0b678fb32d60fc1219c784d6a826fe385aeb79a"
   integrity sha512-9IK9EadsbHo6jLWIpxpR6pL0sazTXV6+SQv25ZB+F7Bj9mJNaOc4nCRabwd5M/JwmUa8idz6Eci6eKfJryPs6Q==
 
-abort-controller@^3.0.0:
-  version "3.0.0"
-  resolved "https://registry.yarnpkg.com/abort-controller/-/abort-controller-3.0.0.tgz#eaf54d53b62bae4138e809ca225c8439a6efb392"
-  integrity sha512-h8lQ8tacZYnR3vNQTgibj+tODHI5/+l06Au2Pcriv/Gmet0eaj4TwWH41sO9wnHDiQsEj19q0drzdWdeAHtweg==
-  dependencies:
-    event-target-shim "^5.0.0"
-
 accepts@~1.3.4, accepts@~1.3.5, accepts@~1.3.7:
   version "1.3.7"
   resolved "https://registry.yarnpkg.com/accepts/-/accepts-1.3.7.tgz#531bc726517a3b2b41f850021c6cc15eaab507cd"
@@ -7268,11 +7261,6 @@ etag@~1.8.1:
   resolved "https://registry.yarnpkg.com/etag/-/etag-1.8.1.tgz#41ae2eeb65efa62268aebfea83ac7d79299b0887"
   integrity sha1-Qa4u62XvpiJorr/qg6x9eSmbCIc=
 
-event-target-shim@^5.0.0:
-  version "5.0.1"
-  resolved "https://registry.yarnpkg.com/event-target-shim/-/event-target-shim-5.0.1.tgz#5d4d3ebdf9583d63a5333ce2deb7480ab2b05789"
-  integrity sha512-i/2XbnSz/uxRCU6+NdVJgKWDTM427+MqYbkQzD321DuCQJUqOuJKIA0IM2+W2xtYHdKOmZ4dR6fExsd4SXL+WQ==
-
 event-to-promise@0.8.0:
   version "0.8.0"
   resolved "https://registry.yarnpkg.com/event-to-promise/-/event-to-promise-0.8.0.tgz#4b84f11772b6f25f7752fc74d971531ac6f5b626"