Skip to content

chore: quick-win triage — global.json, test tooling, localization tests#124

Merged
shanselman merged 3 commits intomasterfrom
chore/quick-wins-triage
Mar 31, 2026
Merged

chore: quick-win triage — global.json, test tooling, localization tests#124
shanselman merged 3 commits intomasterfrom
chore/quick-wins-triage

Conversation

@shanselman
Copy link
Copy Markdown
Collaborator

Three quick wins from Repo Assist triage:

1. Pin .NET SDK (closes #123)

Adds \global.json\ with
ollForward: latestFeature\ to prevent accidental .NET 11 jumps.

2. Sync test tooling (closes #90)

  • xunit.runner.visualstudio 3.1.0 → 3.1.4 in Tray.Tests (matches Shared.Tests)
  • Add coverlet.collector 6.0.4 to Tray.Tests (matches Shared.Tests)

3. Localization validation tests (closes #70)

  • \AllLocales_HaveExactlySameKeysAsEnUs\ — catches missing/extra keys
  • \AllLocales_PreserveFormatPlaceholders\ — prevents runtime FormatException from bad translations

All 598 tests pass (503 shared + 95 tray).

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

shanselman and others added 3 commits March 31, 2026 14:02
…n tests

- Add global.json to pin .NET SDK 10.0.x (rollForward: latestFeature) (closes #123)
- Sync xunit.runner.visualstudio 3.1.0→3.1.4, add coverlet.collector to Tray.Tests (closes #90)
- Add LocalizationValidationTests: key parity + format placeholder validation (closes #70)

All 598 tests pass (503 shared + 95 tray).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
latestFeature picks up SDK 10.0.201 on CI runners, which requires
MSBuild 18.0. The MSIX build step uses VS MSBuild 17.14.x, causing
build failures. latestPatch keeps us in 10.0.1xx where MSBuild 17.x
is compatible.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
ARM64 runner only has 10.0.101, not 10.0.102. Using 10.0.100 as base
with latestPatch finds any 10.0.1xx SDK (101 on ARM64, 104 on x64).
Stays within the feature band where MSBuild 17.x is compatible.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@shanselman shanselman merged commit de51592 into master Mar 31, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant