Skip to content

Exclude the SQLite corpus from the crate package - #394

Merged
KKould merged 1 commit into
mainfrom
chore/exclude-package-files
Oct 26, 2026
Merged

KKould merged 1 commit into
mainfrom
chore/exclude-package-files

Conversation

@KKould

@KKould KKould commented Oct 26, 2026

Copy link
Copy Markdown
Member

What problem does this PR solve?

cargo package -p kite_sql includes the tests/kitesql-testing submodule (54MB SQLite corpus), which exceeds the crates.io 10MB limit and should not be published.

Issue link:

What is changed and how it works?

Add exclude = ["tests/kitesql-testing", "tpcc/results"] to the kite_sql package.

Code changes

  • Has Rust code change
  • Has CI related scripts change

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Note for reviewer

cargo package -p kite_sql --no-verify: 537 files, 5.0MiB (1.6MiB compressed); no corpus or tpcc result files in --list.

@KKould
KKould merged commit 9235a48 into main Oct 26, 2026
12 checks passed
@KKould KKould self-assigned this Oct 26, 2026
@codecov

codecov Bot commented Oct 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.39%. Comparing base (a85b7d3) to head (ddbde8d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #394   +/-   ##
=======================================
  Coverage   93.39%   93.39%           
=======================================
  Files         272      272           
  Lines       39089    39089           
=======================================
  Hits        36508    36508           
  Misses       2581     2581           
Flag Coverage Δ
rust 93.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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