🤖 Merge 'main' => 'xcode26.6'#25845
Draft
github-actions[bot] wants to merge 224 commits into
Draft
Conversation
…1: Build ID 14077114 (#25406) This is the pull request automatically created by the OneLocBuild task in the build process to check-in localized files generated based upon translation source files (.lcl files) handed-back from the downstream localization pipeline. If there are issues in translations, visit https://aka.ms/icxLocBug and log bugs for fixes. The OneLocBuild wiki is https://aka.ms/onelocbuild and the localization process in general is documented at https://aka.ms/AllAboutLoc.
Code Radiator failed on `main` scheduled execution because safe outputs
assumed PR-trigger context and rejected dynamic base overrides. This updates
workflow constraints so scheduled runs can safely update existing merge PRs
and create new ones across target branches.
- **Safe output policy updates**
- Enabled per-run PR base override for expected branch families:
- `safe-outputs.create-pull-request.allowed-base-branches: ["net*.0", "xcode*"]`
- Switched PR-branch push targeting from implicit triggering PR to explicit any-PR mode:
- `safe-outputs.push-to-pull-request-branch.target: "*"`
- Added push guardrail to restrict eligible PRs:
- `safe-outputs.push-to-pull-request-branch.title-prefix: "🤖 Merge 'main' => '"`
- **Checkout behavior aligned to safe-output targeting**
- Added wildcard ref fetch config required for `target: "*"` PR-branch pushes:
- `checkout.fetch: ["*"]`
- `checkout.fetch-depth: 0`
- **Compiled workflow artifact refresh**
- Regenerated `.github/workflows/code-radiator.lock.yml` from updated source workflow frontmatter.
```yaml
checkout:
fetch: ["*"]
fetch-depth: 0
safe-outputs:
create-pull-request:
max: 10
allowed-base-branches:
- "net*.0"
- "xcode*"
push-to-pull-request-branch:
max: 10
target: "*"
title-prefix: "🤖 Merge 'main' => '"
```
---------
Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… TestNSurlSessionHandlerCookieContainerSetCookie (#25420) - [x] Fix `TestNSurlSessionHandlerCookieContainerSetCookie` to treat missing expected cookie as transient network failure (ignore in CI) - [x] Address race condition: move `CookieContainer` inspection after `!completed` guard to avoid concurrent access - [x] Address exception masking: separate `!completed` and `!hasExpectedCookie` CI-ignore guards so real exceptions still fail the test Fixes #25385. --------- Co-authored-by: rolfbjarne <249268+rolfbjarne@users.noreply.github.com>
…5373. (#25399) Use a ConditionalWeakTable to store the additional native memory we need for each NSObject. A bit of testing seems to indicate entries aren't removed from ConditionalWeakTable until they're deallocated (or at the very least can't be resurrected), however a value in the ConditionalWeakTable are *finalized* as soon as the corresponding key is finalizable, which means we need to do some shenanigans to make it work: * We create a class to store the native memory we want to use. * We create one instance of this class for each NSObject instance, and add it to a static ConditionalWeakTable. * In this class' finalizer we free the native memory, but only if the NSObject has been garbage collected. * We determine whether the NSObject has been garbage collected by creating a weak resurrection-tracking GCHandle to the NSObject, and if we determine the NSObject hasn't been garbage collected, we re-schedule the finalizer. This hopefully fixes #25373, where for some reason (I wasn't able to reproduce the problem unfortunately), live NSObject instances ended up with a null pointer for the additional native memory. Fixes #25373.
…20260508175345648 to main (#25377) LEGO: Pull request from lego/hb_5df43909-4a19-4f55-bc3f-9ea8fccf3c82_20260508175345648 to main with localized lcls
Add @media (prefers-color-scheme: dark) rules to xharness.css with dark background, light text, and appropriate link/nav colors. Replace hardcoded 'black' color values with 'currentcolor' so text remains visible in both light and dark modes. Use CSS system color 'Canvas' for nav dropdown background instead of hardcoded #ffffff. --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
#25415) When a method has generic parameter types (like 'T' in Callback<T>), the linker XML descriptor can't resolve the generic parameter name in the method signature. Use the method 'name' attribute instead of 'signature' for such methods, which matches by name and avoids the IL2009 warning. Fixes this warning, which shows up in MAUI's tests (and our own too, just not in a place where we were validating warnings): > warning IL2009: Could not find method 'System.Void Activated(T)' on type 'UIKit.UIGestureRecognizer.Callback`1' --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…20260513055157079 to main (#25414) LEGO: Pull request from lego/hb_5df43909-4a19-4f55-bc3f-9ea8fccf3c82_20260513055157079 to main with localized lcls
…in cookie handler tests (#25419) `TestNSUrlSessionHandlerCookies` and `TestNSUrlSessionEphemeralHandlerCookies` were flaky across unrelated PRs because the CI-ignore guard only checked whether `Set-Cookie` headers were *present*, not whether they contained the expected value. When `httpbin.org` transiently returns a different cookie (e.g. from a CDN/load balancer), `managedCookieResult` is `true` but the value assertion fails — escaping the guard entirely. ## Changes - **`tests/monotouch-test/System.Net.Http/MessageHandlers.cs`** — In `TestNSUrlSessionHandlerCookiesImpl`, pre-compute `managedHasExpectedCookie` / `nativeHasExpectedCookie` after the async block and fold them into the `IgnoreInCI` condition alongside the existing presence checks: ```csharp // Before: only guarded on header presence if (!completed || !managedCookieResult || !nativeCookieResult) TestRuntime.IgnoreInCI ("Transient network failure - ignore in CI"); // After: also guard on expected cookie value var managedHasExpectedCookie = managedCookies?.Any (v => v.StartsWith ("cookie=chocolate-chip;", StringComparison.Ordinal)) == true; var nativeHasExpectedCookie = nativeCookies?.Any (v => v.StartsWith ("cookie=chocolate-chip;", StringComparison.Ordinal)) == true; if (!completed || !managedCookieResult || !nativeCookieResult || !managedHasExpectedCookie || !nativeHasExpectedCookie) TestRuntime.IgnoreInCI ("Transient network failure - ignore in CI"); ``` The downstream assertions reuse the pre-computed booleans, keeping local runs semantically identical. --------- Co-authored-by: rolfbjarne <249268+rolfbjarne@users.noreply.github.com>
…ithin Visual Studio. (#25417) Keep using the desktop MSBuild assemblies when building from within Visual Studio, while we track down why it's not working and fix it for a future release. Ref: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2968169 Ref: #25418 Backport of #25249.
…20260513174343331 to main (#25423) LEGO: Pull request from lego/hb_5df43909-4a19-4f55-bc3f-9ea8fccf3c82_20260513174343331 to main with localized lcls
…20260514054804289 to main (#25425) LEGO: Pull request from lego/hb_5df43909-4a19-4f55-bc3f-9ea8fccf3c82_20260514054804289 to main with localized lcls
) Code Radiator failed when attempting to emit `create_pull_request` for `xcode26.5` because safe outputs only allowed `xcode[0-9]*` (no dotted minor form). This updates the workflow policy so existing branch selection rules and safe-output enforcement are aligned. - **Safe-output base branch policy** - Added dotted Xcode branch support to `create-pull-request.allowed-base-branches` in `.github/workflows/code-radiator.md`. - New allowed pattern: `xcode[0-9]*.[0-9]*`. - **Compiled workflow lock update** - Regenerated `.github/workflows/code-radiator.lock.yml` so runtime safe-output config includes the new pattern. - **Resulting policy shape** - Supports both major and minor Xcode branch naming: - `xcode26` - `xcode26.5` ```yaml safe-outputs: create-pull-request: allowed-base-branches: - "net[0-9]*.0" - "xcode[0-9]*" - "xcode[0-9]*.[0-9]*" ``` --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: rolfbjarne <249268+rolfbjarne@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
This pull request updates the following dependencies ## From https://github.com/dotnet/xharness - **Subscription**: [02e03784-16b3-4ced-b02a-3715797fc7da](https://maestro.dot.net/subscriptions?search=02e03784-16b3-4ced-b02a-3715797fc7da) - **Build**: [20260514.1](https://dev.azure.com/dnceng/internal/_build/results?buildId=2974748) ([314304](https://maestro.dot.net/channel/2/github:dotnet:xharness/build/314304)) - **Date Produced**: May 14, 2026 9:16:56 AM UTC - **Commit**: [51ca379106cfd749a498cb0822210ef1aa926e41](dotnet/xharness@51ca379) - **Branch**: [main](https://github.com/dotnet/xharness/tree/main) - **Dependency Updates**: - From [11.0.0-prerelease.26230.4 to 11.0.0-prerelease.26264.1][1] - Microsoft.DotNet.XHarness.iOS.Shared [1]: dotnet/xharness@92962e5...51ca379
Convert classic NUnit assertions (`Assert.AreEqual`, `Assert.IsTrue`, `Assert.IsNotNull`, etc.) to NUnit v4's constraint-based `Assert.That` syntax in the msbuild test projects. Also update package references to use centralized version properties from `Directory.Build.props`: - `NUnit` → `$(NUnitPackageVersion)` - `NUnit3TestAdapter` → `$(NUnit3TestAdapterPackageVersion)` - `Microsoft.NET.Test.Sdk` → `$(MicrosoftNETTestSdkPackageVersion)` Two shared files (`tests/common/DotNet.cs`, `tests/common/mac/ProjectTestHelpers.cs`) are also converted — the `Assert.That` constraint syntax is compatible with both NUnit 3 and 4, so other projects referencing these files are unaffected. All `tests-msbuild` tests pass (495 passed, 0 failed). 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
## Summary Introduces a stack-allocated `ObjCSuper` struct to replace the use of `NSObject.SuperHandle` for `objc_msgSendSuper` calls. This eliminates the need to allocate and manage unmanaged memory for the `objc_super` struct on every super call. ## Changes - **New `ObjCSuper` readonly ref struct** (`src/ObjCRuntime/ObjCSuper.cs`): A `[StructLayout(LayoutKind.Sequential)]` struct with `Receiver` and `ClassHandle` fields matching the native `objc_super` layout. - **Updated bgen code generation**: P/Invoke declarations for `objc_msgSendSuper` variants now take `ObjCSuper*` instead of `IntPtr` as the first parameter. Generated call sites create a stack-allocated `ObjCSuper` and pass `&__objc_super__` instead of `this.SuperHandle`. - **Updated handwritten super calls** in 11 source files (CIFilter, CIVector, CALayer, UIPasteboard, NSInputStream, NSUuid, NSThread, NSHttpCookie, GCMouse, NSOpenGLPixelFormat, NSBitmapImageRep) plus NSObject2.cs. - `NSObject.SuperHandle` is preserved for backward compatibility (can be simplified/removed in XAMCORE_5_0). Fixes #25362 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
## Summary Line 22 of `docs/native-library-interop.md` contained a broken internal link pointing to `build-apps/build-items.md`. The directory `docs/build-apps/` does not exist in the repository; the correct path is `docs/building-apps/build-items.md`. ## Changes - Fixed the link target on line 22: `build-apps/build-items.md` → `building-apps/build-items.md` ## Verification - `docs/building-apps/build-items.md` exists in the repo ✅ - The corrected line reads `[build-items](building-apps/build-items.md)` ✅ --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…25442) Convert classic NUnit assertions to NUnit v4's constraint-based Assert.That syntax in the introspection test project. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…macios (#25441) All 11 numbered code reference links in `docs/bindas.md` (links [1]–[7], [9]–[12]) pointed to a personal fork (`rolfbjarne/xamarin-macios`) at a specific commit SHA. These links are not guaranteed to remain accessible. Link [8] (a PR link) was already correct. This PR updates all 11 links to use canonical `dotnet/macios` permalinks with the current main branch SHA (`18a22fd`), with corrected line numbers matching the current state of the codebase. ### Changes - **Base URL**: `rolfbjarne/xamarin-macios/blob/b38c114f…` → `dotnet/macios/blob/18a22fd0…` - **Links [11]/[12]**: path corrected from `src/generator.cs` → `src/bgen/Generator.cs` (file was renamed) - **All line numbers**: updated to match the current tree (e.g., `testgenerator.cs#L100` → `#L125`, `Registrar.cs#L687` → `#L794`) ### Verification All 11 updated URLs return HTTP 200. This is a documentation-only change with no code impact. --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
) The DetectMetadata tests were flaky because the macOS pasteboard subsystem may not have fully analyzed content metadata by the time DetectMetadata is called immediately after SetStringForType. The callback would be invoked but return an empty dictionary. Fix by: - Adding retry logic (up to 5 attempts with 500ms delays) for the '#2' detection case where we expect results after writing content - Increasing WaitOne timeouts from 1s to 10s for CI under load Fixes #25446 --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
This pull request updates the following dependencies ## From https://github.com/dotnet/macios - **Subscription**: [c0371266-dd6f-4959-822b-decc72d2d668](https://maestro.dot.net/subscriptions?search=c0371266-dd6f-4959-822b-decc72d2d668) - **Build**: [20260518.1](https://dev.azure.com/devdiv/DevDiv/_build/results?buildId=14122183) ([314711](https://maestro.dot.net/channel/3884/github:dotnet:macios/build/314711)) - **Date Produced**: May 18, 2026 8:31:49 AM UTC - **Commit**: [7075bb3](7075bb3) - **Branch**: [release/9.0.1xx](https://github.com/dotnet/macios/tree/release/9.0.1xx) - **Dependency Updates**: - From [26.5.9001 to 26.5.9002][1] - Microsoft.iOS.Sdk.net9.0_26.5 - Microsoft.MacCatalyst.Sdk.net9.0_26.5 - Microsoft.macOS.Sdk.net9.0_26.5 - Microsoft.tvOS.Sdk.net9.0_26.5 [1]: 1e1deb8...7075bb3
* Add a 'StringUtils.IsNullOrEmpty' with proper nullability attributes, because the netstandard2.0 version doesn't have it. * Add a 'string.EndsWith(char)' overload. --------- Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
Useful when: * Building on Linux. * Building on older macOS versions, that can't install the required Xcode version. Refactor the existing IS_LINUX variable to use NO_XCODE instead.
#25452) Expand IgnoreInCIIfSshConnectionError to also detect 'A TLS error caused the secure connection to fail' messages and CFNetworkErrors.SecureConnectionFailed NSError codes by walking the entire exception chain. Add IgnoreInCIIfBadNetwork(ex) calls to tests that were only checking HTTP status codes but not exceptions: - GHIssue8342 - SupportsDigestAuthentication - GHIssue16339 - AcceptSslCertificatesServicePointManager Fixes #25444 --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
…atic registrars (#25434) * Fix an issue where the new style of generated block supporting code wasn't properly handled by the trimmer when using the dynamic registrar. * Add a 'link sdk' + 'link all' test variation for all platforms for monotouch-test. * Update a few tests to cope with the additional trimming variations for monotouch-test. --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
The red checkmark should be enough, and this way we don't require write permissions to the pull request.
…25749) Replace the Makefile-based test runner (`packaged-macos-tests.mk` + `run-with-timeout` + `mac-test-report-generator`) with a single C# script project (`scripts/run-packaged-macos-tests`) that: - Executes pre-built macOS and Mac Catalyst test apps with timeout and launch retry logic - Generates an HTML report with per-platform/architecture breakdown tables showing output links and [FAIL] lines inline - Generates TestSummary.md for GitHub PR comments - Handles ARM64/x64 architecture detection and skipping - Collects and links crash reports Simplifies `run_mac_tests.ps1` from ~235 lines to ~150 lines by delegating all test execution and reporting to the new C# tool. 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ion. Fixes #25667. (#25699) Surface cancelled requests with an `TaskCanceledException` instead of a `TimeoutException`. This makes it easier for consumers to determine what happened. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> 🤖 Pull request created by Copilot Fixes: #25667 --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
Improve XML documentation for the `MPSkipIntervalCommand` type: - Replace 'To be added.' placeholders with meaningful descriptions - Fix `see cref` reference to use unqualified `MPRemoteCommandCenter` - Remove empty `<remarks>` nodes - Add proper documentation for the `PreferredIntervals` property 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ts (#25794) --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… reference output paths (#25790) A binding `.resources` sidecar manifest is passive data that may come from a restored package, but `ResolveNativeReferences` copied every manifest element into item metadata. Build-controlled path/identity metadata such as `RelativePath`, `ReidentifiedPath` or `DynamicLibraryId` could then flow into the native-library output path and make `InstallNameTool` create directories and write files outside the intended intermediate output directory. This PR: - Filters the metadata copied from a manifest using a **block-list** of the build-controlled path/layout/identity sinks (`RelativePath`, `ReidentifiedPath`, `ComputedRelativePath`, `DynamicLibraryId`, `PublishFolderType`, `TargetDirectory`, `SourceDirectory`), warning when one is dropped. Everything else — including binding-defined metadata like `NoDSymUtil`/`NoSymbolStrip` — is legitimate native-reference content and is copied as-is. - Skips native references with no name, rejects names containing rooted/`..` traversal segments, and ignores reserved MSBuild metadata names (e.g. `FullPath`) instead of crashing the task. - Adds a `PathUtils.IsPathContained` containment check so a reidentified native library can never be written outside the intended output directory, and deletes the temporary file if `install_name_tool` fails.
… bace0c94c5ec24d81dc66d1adfb5ce44edb218e6 to 0ee033f311c9e1aca360c18e0c3f7c2c61b7f2be (#25800) Bumps [dotnet/arcade/.github/workflows/inter-branch-merge-base.yml](https://github.com/dotnet/arcade) from bace0c94c5ec24d81dc66d1adfb5ce44edb218e6 to 0ee033f311c9e1aca360c18e0c3f7c2c61b7f2be.
Improve XML documentation for the `MPVolumeSettings` type: - Replace 'To be added.' placeholders with meaningful descriptions - Remove empty `<remarks>` nodes - Add proper documentation for `AlertShow`, `AlertHide`, and `AlertIsVisible` methods 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…'. (#25806) And recompile. This should eventually get rid of the COPILOT_GITHUB_TOKEN secret, once github/gh-aw#38900 is released (and we update and recompile with that version of gh-aw).
…e condition) (#25797) Run 'dotnet --info' once right after downloading the .NET SDK to complete the first-time NuGet migrations logic. This avoids the race condition where multiple parallel dotnet processes attempt the first-time setup simultaneously, causing 'The system cannot open the device or file specified' errors. The workaround only runs in CI (when the ACES environment variable is set). Ref: dotnet/runtime#91987 --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Improve XML documentation for the `MPNowPlayingInfo` type: - Replace 'To be added.' placeholders with meaningful descriptions - Remove empty `<remarks>` nodes - Add proper documentation for all properties (title, artist, album, artwork, duration, elapsed time, playback rate, etc.) 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Below the top-level 'updates' entry, there should be only the 'package-ecosystem' entry, all other entries should be under that 'package-ecosystem' entry. This hopefully fixes a problem with dependabot where dependabot shouldn't update compiled agentic workflow *.lock.yml files.
And disable it for CI. --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Rolf Bjarne Kvinge <rokvin@microsoft.com>
Add fallback logic so that when the create_pull_request safeoutput tool fails, the workflow files a GitHub issue to track the failure instead of silently dropping it. This ensures merge failures are visible and can be resolved manually. Changes: - Add create-issue safe-output (max: 10) - Add fallback section (step 2f) describing issue creation on failure - Update summary to report PR creation failures with issue links --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…25785) The `[comment]: <> (...)` identifier string is used to detect and hide older GitHub comments of the same type during CI. This string was added at the end of each comment, which means that when a comment was too large for GitHub and got truncated, the identifier was cut off. This prevented the comment from being detected and hidden in the future. Move the identifier to the beginning of the comment so it is never lost to truncation. Changes: - `GitHub.psm1`: Added `WriteCommentIdentifier()` method, called at the top of all three `NewComment*` methods. Removed identifier from `WriteCommentFooter`. - `TestResults.psm1`: Moved the `[comment]: <>` line from end to beginning in both code paths of `WriteComment()`. - `TestResults.Tests.ps1`: Updated all 5 affected test expectations. 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Improve XML documentation for the `MPNowPlayingInfoCenter` type: - Replace 'To be added.' placeholders with meaningful descriptions - Remove empty `<remarks>` nodes - Improve class summary to be more concise - Add proper documentation for the `NowPlaying` property 🤖 Pull request created by Copilot --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
….yml from 0ee033f311c9e1aca360c18e0c3f7c2c61b7f2be to b076228a542025c4f879f254d38adb5cf34a2475 (#25817) Bumps [dotnet/arcade/.github/workflows/inter-branch-merge-base.yml](https://github.com/dotnet/arcade) from 0ee033f311c9e1aca360c18e0c3f7c2c61b7f2be to b076228a542025c4f879f254d38adb5cf34a2475.
We're not using it anymore.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
There was a problem hiding this comment.
Protected files were modified in this pull request and require manual scrutiny before merge.
Please verify that each protected-file change is intentional, policy-compliant, and safe:
- Protected files:
global.json,README.md,.github/workflows/bump-global-json.yml
Collaborator
Collaborator
🔥 [PR Build #6894ac2] Build failed (Build packages) 🔥Build failed for the job 'Build packages' (with job status 'Failed') Pipeline on Agent |
Contributor
Author
|
⏭️ Skipping merge update: this PR is a draft. Convert to ready when you want automated updates to resume.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Caution
Protected files were modified in this change.
This pull request is in
request_reviewmode and requires explicit human scrutiny before merge.Protected files:
global.json,README.md,.github/workflows/bump-global-json.ymlAutomated merge of
mainintoxcode26.6.Created by the code-radiator workflow.