Skip to content

Run rusqlite's SQLite3MC tests on the vendored amalgamation in CI - #10

Merged
LucaCappelletti94 merged 3 commits into
mainfrom
upstream/ci-rusqlite-consumer
Sep 30, 2026
Merged

LucaCappelletti94 merged 3 commits into
mainfrom
upstream/ci-rusqlite-consumer

Conversation

@LucaCappelletti94

Copy link
Copy Markdown
Owner

sqlite3mc-src exists to feed the bundled-sqlite3mc feature of libsqlite3-sys, yet CI never built that consumer on the bytes under review. A SQLite3MC release that drops or renames a function the committed rusqlite bindings declare would pass every check here and break only when someone built rusqlite.

The new rusqlite job fetches the feat/sqlite3mc-src branch of LucaCappelletti94/rusqlite at one pinned commit, points its sqlite3mc-src dependency at this checkout and runs rusqlite's tests with the feature sets that branch runs, including a buildtime_bindgen pass that regenerates the bindings from the shipped header. The branch requires one exact sqlite3mc-src release, and Cargo silently drops a patch whose version misses it, so the job rewrites that requirement to the checkout's version and ends by checking that the build output names this checkout's sqlite3mc directory. Without both, a release-bot PR would test the old crates.io bytes and pass.

A weekly rusqlite pin workflow goes red when the fork branch moves past the pinned commit or when a released rusqlite gains bundled-sqlite3mc, so the pin moves to a released coordinate as soon as one exists.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository: LucaCappelletti94/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: b9266749-0e3d-447a-bd69-39c46ef36798

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (2e8b059) to head (663e653).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            3         3           
  Branches         3         3           
=========================================
  Hits             3         3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LucaCappelletti94
LucaCappelletti94 force-pushed the upstream/ci-rusqlite-consumer branch from 8b521a1 to 663e653 Compare September 30, 2026 15:20
@sonarqubecloud

Copy link
Copy Markdown

@LucaCappelletti94
LucaCappelletti94 merged commit a42c90d into main Sep 30, 2026
22 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

Development

Successfully merging this pull request may close these issues.

1 participant