Skip to content

mv c/sqlite => github.com/llarhub/sqlite - #61

Merged
xushiwei merged 2 commits into
goplus:mainfrom
xushiwei:t
Oct 26, 2026
Merged

xushiwei merged 2 commits into
goplus:mainfrom
xushiwei:t

Conversation

@xushiwei

Copy link
Copy Markdown
Member

No description provided.

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: Remove c/sqlite package

This PR is a clean, pure deletion that removes the entire c/sqlite package (3 files, 355 deletions, 0 additions), relocating it to github.com/llarhub/sqlite per the commit message.

Verification performed (via automated code-quality, performance, security, and documentation-accuracy passes):

  • No source file imports or references github.com/goplus/lib/c/sqlite — the deletion breaks no builds.
  • The c/sqlite/ directory is fully removed; no stray files remain.
  • No docs index, top-level README, CHANGELOG, go.mod, c/llcppg.pub, or CI config references the package.
  • No secrets, trust-boundary, or performance impact from the removal.
  • Remaining sqlite matches in py/pandas/gen.go are unrelated generated pandas docstrings about Python's sqlite3 module.

No blocking issues found. The removal is complete and safe.

Optional, pre-existing (out of scope — not introduced by this PR):

  • c/libuv/README.md:42 still shows the example cd _demo/sqlitedemo, but libuv has no such demo (its demos are async_fs / echo_server). This is a stale copy-paste artifact that predates this PR. Now that the repo's only sqlitedemo is being removed, you may want to fix that example to a real libuv demo in a separate change.

@xushiwei
xushiwei merged commit d427da6 into goplus:main Oct 26, 2026
5 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