Skip to content

build(deps): analyzer and SourceLink bumps in Directory.Build.props (supersedes #623, part 3 of 3) - #626

Merged
Chris-Wolfgang merged 2 commits into
deps/supersede-623-local-toolsfrom
deps/supersede-623-analyzers
Nov 2, 2026
Merged

Chris-Wolfgang merged 2 commits into
deps/supersede-623-local-toolsfrom
deps/supersede-623-analyzers

Conversation

@Chris-Wolfgang

@Chris-Wolfgang Chris-Wolfgang commented Oct 31, 2026 •

Copy link
Copy Markdown
Owner

Summary

Supersedes the protected half of Dependabot #623: the three bumps in Directory.Build.props. Stacked on #625: merge #583 -> #614 -> #615 -> #624 -> #625 first. Together, #625 and this PR carry all 14 updates from #623.

Bumps (same versions as #623):

  • Microsoft.VisualStudio.Threading.Analyzers 18.7.23 -> 18.12.1
  • Meziantou.Analyzer 3.0.296 -> 3.0.318
  • Microsoft.SourceLink.GitHub 10.0.401 -> 10.0.403

Protected-file PR (Directory.Build.props only).

Verification

Verified locally; CI blocked by the account billing lock.

  • dotnet build "ETL Fixed Width.slnx" -c Release --no-incremental (all projects, all TFMs, TreatWarningsAsErrors): exit 0, 0 errors, 1 warning (the pre-existing CS0618 in examples/Metrics/Program.cs:74, also present without this change). The new analyzer versions add no diagnostics.
  • Projects outside the solution, Release: samples/ShadowWorkloads and tools/GcProfile 0 warnings; tests/AotSmoke 7 warnings (CA1508/MA0006 in Program.cs), the same 7 that ci(aot): adopt the canonical AOT smoke workflow #619 reports as pre-existing.
  • Tests were not re-run for this PR (analyzer and SourceLink versions only); build(deps): NuGet package bumps in the csproj files (supersedes #623, part 1 of 3) #625 below it ran the full build-pr.ps1 gate: 786/786 tests, line 99.2%, gate passed.

After this merges, close #623 by hand (the burn-down never closes PRs).

Dependency stack

Position 7 of 7. Below: #631. Above: none (TOP of the stack).

Merge order: #583 → #614 → #615 → #624 → #625 → #631 → #626

Merge bottom-up, and merge this PR only after its base shows main. Branches are kept current by merging the base in (no force-pushes), so merge commits in this branch are expected.

🤖 Generated with Claude Code

….props)

- Microsoft.VisualStudio.Threading.Analyzers 18.7.23 -> 18.12.1
- Meziantou.Analyzer 3.0.296 -> 3.0.318
- Microsoft.SourceLink.GitHub 10.0.401 -> 10.0.403

Protected file, so its own PR (completes the supersede of #623).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@Chris-Wolfgang
Chris-Wolfgang changed the base branch from deps/supersede-623-nuget-packages to deps/supersede-623-local-tools November 2, 2026 02:22
@Chris-Wolfgang Chris-Wolfgang changed the title build(deps): analyzer and SourceLink bumps in Directory.Build.props (supersedes #623, part 2 of 2) build(deps): analyzer and SourceLink bumps in Directory.Build.props (supersedes #623, part 3 of 3) Nov 2, 2026
@Chris-Wolfgang
Chris-Wolfgang added this pull request to stack #643 November 2, 2026 15:23
@Chris-Wolfgang
Chris-Wolfgang merged commit 0cfca51 into main Nov 2, 2026
23 of 24 checks passed
@Chris-Wolfgang
Chris-Wolfgang deleted the deps/supersede-623-analyzers branch November 2, 2026 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant