Bump streetsidesoftware/cspell-action from 8.4.0 to 9.0.1 in the actions-deps group - #444
Closed
dependabot[bot] wants to merge 1 commit into
Closed
Conversation
Bumps the actions-deps group with 1 update: [streetsidesoftware/cspell-action](https://github.com/streetsidesoftware/cspell-action). Updates `streetsidesoftware/cspell-action` from 8.4.0 to 9.0.1 - [Release notes](https://github.com/streetsidesoftware/cspell-action/releases) - [Changelog](https://github.com/streetsidesoftware/cspell-action/blob/main/CHANGELOG.md) - [Commits](streetsidesoftware/cspell-action@de2a73e...e0668cf) --- updated-dependencies: - dependency-name: streetsidesoftware/cspell-action dependency-version: 9.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions-deps ... Signed-off-by: dependabot[bot] <support@github.com>
This was referenced Aug 29, 2026
ptr727 added a commit
that referenced
this pull request
Aug 29, 2026
## Why The .NET 10 SDK dropped the VSTest bridge `dotnet test` relied on, so `dotnet test` failed outright on every PR: `Testing with VSTest target is no longer supported by Microsoft.Testing.Platform on .NET 10 SDK and later.` This is what was blocking #443, #444, #445, and #446 (all four currently fail on the `Run unit tests job`). ## What - Opt `UtilitiesTests` into native Microsoft.Testing.Platform (MTP): `global.json`'s `test.runner` setting, plus `UseMicrosoftTestingPlatformRunner`/`OutputType=Exe` on the test project, per the [official migration guide](https://learn.microsoft.com/en-us/dotnet/core/testing/migrating-vstest-microsoft-testing-platform). - Swap `coverlet.collector` (VSTest-only) for `Microsoft.Testing.Extensions.CodeCoverage`, the native MTP coverage provider. - Bump `Microsoft.NET.Test.Sdk` and the `xunit.v3` family to the versions that ship a compatible MTP runtime; the prior `xunit.v3` 3.2.2 pairing threw a `TypeLoadException` against the newer platform assembly. - Update the validate workflow's `dotnet test` invocation (`--coverage` instead of `--collect`), naming the output file explicitly (`--coverage-output coverage.cobertura.xml`): the extension's default GUID basename is not matched by codecov-action's file finder, so the upload step would otherwise silently find nothing under `fail_ci_if_error: false`. - Fix three `ExtensionsTests.cs` null-argument tests that were missing the null-forgiving operator a sibling test already used; `TreatWarningsAsErrors` never reached these under the old VSTest error, which aborted the build before the test project ever compiled. - Add `global.json` to the Solution Items folder and fix `Directory.Packages.props`'s alphabetical ordering. ## Verification Ran locally against .NET 10.0.400: `dotnet build` (0 warnings/errors), `dotnet test --coverage --coverage-output-format cobertura --coverage-output coverage.cobertura.xml --results-directory ./coverage` (183/183 passed, `coverage/coverage.cobertura.xml` produced), `dotnet csharpier check .`, and `dotnet format style --verify-no-changes` all clean. Reviewed with a local adversarial pass before pushing (fleet `local-strict-review`). ## Known trade-off `Microsoft.Testing.Extensions.CodeCoverage` ships native instrumentation for `win-x64/x86/arm64`, `linux-x64`, `linux-musl-x64`, and `osx-x64` only, no `osx-arm64` or `linux-arm64`. CI runs on `ubuntu-latest` (x64) and is unaffected, but coverage collection won't work locally on Apple Silicon or Linux arm64 dev machines, where `coverlet.collector` had none of that restriction. Flagging for awareness rather than blocking on it, since this is the officially recommended MTP coverage path. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Updated the test runner and testing tools for improved compatibility and execution. * Added consistent Cobertura code coverage reporting. * Preserved validation of expected compression and decompression errors. * **Chores** * Standardized .NET SDK and test environment configuration. * Updated test tooling and coverage integration packages. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
ptr727 added a commit
that referenced
this pull request
Aug 29, 2026
## Why Same regression as #447, on `main` this time (per `.github/dependabot.yml`, `main` and `develop` are kept current independently). The .NET 10 SDK dropped the VSTest bridge `dotnet test` relied on, so `dotnet test` failed outright: `Testing with VSTest target is no longer supported by Microsoft.Testing.Platform on .NET 10 SDK and later.` This is what was blocking #443 and #444 (both currently fail on the `Run unit tests job`). ## What Identical fix to #447, cherry-picked (the relevant files were byte-identical between `main` and `develop` before this PR): - Opt `UtilitiesTests` into native Microsoft.Testing.Platform (MTP) via `global.json`'s `test.runner` setting plus `UseMicrosoftTestingPlatformRunner`/`OutputType=Exe`. - Swap `coverlet.collector` (VSTest-only) for `Microsoft.Testing.Extensions.CodeCoverage` (native MTP coverage), marked test-only via `PrivateAssets`. - Bump `Microsoft.NET.Test.Sdk`/`xunit.v3`/`xunit.analyzers`/`xunit.runner.visualstudio` to versions with a compatible MTP runtime. - Update the validate workflow's `dotnet test` invocation (`--coverage` instead of `--collect`, with an explicit `--coverage-output` filename codecov-action can discover). - Fix three `ExtensionsTests.cs` null-argument tests missing a null-forgiving operator. - `global.json` in CRLF (this repo's `.editorconfig` convention) and added to Solution Items. ## Verification Already went through #447's full review loop (local adversarial review, Copilot, CodeRabbit, all findings fixed) on identical content. Re-verified independently on this branch: `dotnet build` (0 warnings/errors), `dotnet test --coverage --coverage-output-format cobertura --coverage-output coverage.cobertura.xml --results-directory ./coverage` (183/183 passed), `dotnet csharpier check .`, `dotnet format style --verify-no-changes`, and `editorconfig-checker` on `global.json` all clean. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Updated automated test execution with a modern test runner for more reliable validation. * Added improved code coverage collection and Cobertura report generation for clearer quality metrics. * Updated testing tools and frameworks to newer versions. * Preserved existing compression test behavior while improving nullable-value handling during test execution. * **Chores** * Added centralized configuration for consistent test tooling across the solution. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
ContributorAuthor
Looks like streetsidesoftware/cspell-action is no longer updatable, so this is no longer needed. |
auto-merge was automatically disabled
August 30, 2026 15:12
Pull request was closed
dependabotBot
deleted the
dependabot/github_actions/develop/actions-deps-adb96f754f
branch
August 30, 2026 15:12
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 freeto 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.
Bumps the actions-deps group with 1 update: streetsidesoftware/cspell-action.
Updates
streetsidesoftware/cspell-actionfrom 8.4.0 to 9.0.1Release notes
Sourced from streetsidesoftware/cspell-action's releases.
Changelog
Sourced from streetsidesoftware/cspell-action's changelog.
... (truncated)
Commits
e0668cfchore(main): release 9.0.1 (#2734)ed61a9efix: make sure the dictionaries are up to date. (#2733)a670ca8chore(main): release 9.0.0 (#2682)7319e02ci: Workflow Bot -- Update ALL Dependencies (main) (#2732)ffbcf8fchore: Set pnpm minimumReleaseAge (#2731)c48ca1cchore: Update PNPM (#2664)c223c96chore(deps-dev): bump tsdown from 0.21.10 to 0.22.14 (#2721)f433a62fix: Workflow Bot -- Update ALL Dependencies (main) (#2710)7a30246fix: Update Dictionaries and Dependencies (#2729)7f801dafix: Update Dictionaries and Dependencies (#2717)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions