Point Dependabot at dev, and sync main back into dev - #409
Merged
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add powered-by line on landing page
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add Darling Data favicon
Uses the Darling Data barbell logo as hero image when shared on social media. Also adds meta description for SEO. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add Open Graph meta tags for social sharing
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Clarify social share description
Merge dev: HTML export for plan analysis (#182)
This reason means something in the query blocks parallelism (scalar UDFs, table variable inserts, etc.) — that's worth a Warning, not Info. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fix Rule 3: CouldNotGenerateValidParallelPlan stays Warning
Adds human-readable messages for all 25 known reasons. Severity: - Warning: actionable reasons (UDFs, cursors, table variables, remote queries, trace flags, hints, DML OUTPUT, writeback variables) - Info: passive/environmental (cost below threshold, edition limits, memory-optimized tables, upgrade mode, index build edge cases) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Expand Rule 3 to cover all NonParallelPlanReason values
Merge dev: plan sharing and export (#182)
Release v1.4.1
Release v1.4.2
Release v1.4.3
Merge dev to main (v1.4.3 + logo fix)
Merge dev: HTTPS plan sharing API
Release: issue #178 round 3 feedback (items 17-25)
Release v1.5.0 — maximum benefit scoring
Release: analytics for Plan Analyzer web app
Merge dev to main
Merge dev to main — Joe #215 feedback through b1/b7
Release v1.6.0
Release v1.7.0 — wait stats as warnings
Release v1.7.1 — render benefit % and actionable fix
Release v1.7.2 — operator self-time fix + decimal benefit %
Release v1.7.3 — visible web viewer version
Release v1.7.4 — external-wait formula + CPU:Elapsed adjustment
Release v1.7.7 — D3 content strip + legacy marker + drop Rule 21
Release v1.7.8 — batch-mode pipelined self-time (#215 D1)
Release v1.8.0 — Joe feedback E1-E12 batch
Release: dev -> main
Release v1.10.0
Release v1.11.0
Release v1.11.1
Release v1.11.2
Release v1.12.0
Release: web parallelism icons, SSMS extension, connect-dialog DB, hardening
Release v1.14.0
Release v1.14.1
Release v1.14.2
Release v1.15.0 - Query Store colorblind-safe palette
Release v1.16.0 - Query Store search and filtering
Release v1.17.0
Release v1.18.0
--- updated-dependencies: - dependency-name: SQLitePCLRaw.bundle_e_sqlite3 dependency-version: 3.0.4 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
…3.0.1 --- updated-dependencies: - dependency-name: Meziantou.Framework.Win32.CredentialManager dependency-version: 3.0.1 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
…nViewer.App/Meziantou.Framework.Win32.CredentialManager-3.0.1 deps: Bump Meziantou.Framework.Win32.CredentialManager from 2.0.1 to 3.0.1
The comment said "remove this once its floor reaches 2.1.12+", which is stale now that the ref is on the 3.x line. Microsoft.Data.Sqlite 10.0.10 still floors its transitive at the vulnerable 2.1.11, so this direct ref is still what keeps the native library patched. Records what was verified: 3.0.4 bundles SQLite 3.53.3 (past the 3.50.2 CVE fix), and Microsoft.Data.Sqlite 10.0.10 works against the 3.x native layer - stats aggregates, insert, select round-trip, parameterized delete, and 404-after-delete all exercised against a running instance. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…PlanShare/SQLitePCLRaw.bundle_e_sqlite3-3.0.4 deps: Bump SQLitePCLRaw.bundle_e_sqlite3 from 2.1.12 to 3.0.4
Two related fixes for a mistake in the original Dependabot config. target-branch was not set, so Dependabot defaulted to the repository's default branch - main. Its first two PRs (#407, #408) were therefore opened against main rather than dev, and merging them put dependency bumps directly onto main, bypassing the dev -> main release flow and leaving dev behind. Both entries now set target-branch: dev, with a comment explaining why it must stay. This branch also merges main back into dev so dev is no longer missing those bumps: SQLitePCLRaw.bundle_e_sqlite3 3.0.4 and Meziantou.Framework.Win32.CredentialManager 3.0.1. Merge was clean. Verified: clean Debug build 0 warnings, 288/288 tests pass. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Reviewed. This is a small, config-only PR (dependabot fix + sync of two already-shipped dependency bumps) — none of the usual plan-XML / T-SQL / PlanViewer.Ssms-version concerns apply here since no code paths for that were touched. Checked and looks correct:
No issues found. The one soft risk (skipped version verification round-trips for the two bumps) is addressed in the PR description rather than in code, which is fine for a config-only change like this. |
This was referenced Jul 25, 2026
Merged
Merged
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.
Fixes a mistake in the Dependabot config I added in #399, and repairs its consequence.
What went wrong
.github/dependabot.ymldid not settarget-branch. Dependabot therefore defaults to the repository's default branch, which here ismain.So its first two PRs (#407, #408) were opened against
main, notdev. I merged them without checking the base, which:main, bypassing thedev->mainrelease flowdevbehindmain, which would have surfaced later as a confusing conflict on the next release PRThat is on me - the base branch is right there in the PR and I did not look at it.
Fix
Both ecosystem entries now set
target-branch: dev, with a comment explaining why it has to stay. Future Dependabot PRs will targetdevand follow the normal flow.Sync
This branch also merges
mainback intodev, sodevis no longer missing the two bumps:SQLitePCLRaw.bundle_e_sqlite33.0.4Meziantou.Framework.Win32.CredentialManager3.0.1The merge was clean - no conflicts.
What was not affected
Both bumps were verified before merging, and that work stands:
SaveCredential/DeleteCredentialswallow exceptions, so a broken major would have compiled and then silently failed to save credentials; the round-trip is what rules that out.And the accidental merge to
maindid have one genuinely good outcome: it triggered the new PlanShare auto-deploy for real. Production is now on SQLite 3.53.3, service active,api/statsreturning 200, binary timestamped 21:49 UTC. The deploy automation is validated end-to-end on its first real trigger.Test plan
dependabot.ymlparses; both entries reporttarget-branch: dev🤖 Generated with Claude Code