Bump version to 1.20.0 - #444
Merged
Merged
Conversation
Minor rather than patch. 1.19.x would understate it: #439 adds a "source" field to every warning in the JSON and MCP output and a new badge in the app and CLI, and #437 changes what an existing analysis rule concludes about a plan. Both are things a consumer can notice, and one of them is output-shape. What ships: - #437 Rule 12 no longer calls a conversion non-SARGable when it converts the parameter rather than the column. Plans carrying a parameter-side conversion on a scan lose that warning and report the scan's residual predicate instead. Verified against all 38 committed plans: no other plan's verdict moves. - #439 SQL Server's own warnings are now told apart from ours, tagged [SQL Server] in the app and CLI and carried as "source" in JSON/MCP. Additive, but it changes the bytes of analyze --compact. - #431 Robot Advice no longer takes the app down on a deep plan. - #438 querystore gets the same depth ceiling analyze got; it had been failing quietly on deep plans, one ERROR row per plan. - #443 the macOS handle resolver no longer corrupts memory on Apple silicon. - #425 Entra MFA works again (WAM parent window handle). Not user-facing but worth knowing for anyone building from this tag: the suite runs on Microsoft.Testing.Platform now (#442), and `dotnet test` finishes on macOS for the first time (#443) - 307 tests, 305 passing, 13 seconds. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Reviewed. Straightforward version bump — verified all four version-bearing spots are consistent: |
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.
Version bump for v1.20.0. Merging this to
devpublishes nothing — the release fires whendevis merged tomain.Why minor, not patch
1.19.2 would understate it. #439 adds a
sourcefield to every warning in the JSON and MCP output and a new badge in the app and CLI, and #437 changes what an existing analysis rule concludes about a plan. Both are noticeable to a consumer, and one of them is output-shape.What ships
User-facing
[SQL Server]in the app and CLI, carried assourcein JSON/MCP. Additive, but it changes the bytes ofanalyze --compact(ExpectedCompactOutputSha256was rolled deliberately).querystoregets the same depth ceilinganalyzegot. It had been failing quietly on deep plans — oneERRORrow insummary.txtper plan — for as long asanalyzehad been fine.Not user-facing, but relevant to anyone building from this tag
xunit.v34.0.0 dropping VSTest on .NET 10).dotnet testfinishes on macOS for the first time: 307 tests, 305 passing, 13 seconds. It previously did not complete at all.Issues this closes on release
#430, #435, #436 — all three have been waiting on a release, and two have reporters who supplied a crash log and a repro script.
Verification
dotnet buildclean (the 9 warnings are the pre-existingMCP9005obsolete-API uses inMcpSmokeTests.cs).dotnet teston macOS ARM64: 307 total, 305 passed, 0 failed, 2 skipped.🤖 Generated with Claude Code