Skip to content

chore(libcalibre): delete dead api.rs#126

Merged
phildenhoff merged 2 commits into
mainfrom
phil/cdl-5-chore-delete-dead-crateslibcalibresrcapirs
Jun 14, 2026
Merged

chore(libcalibre): delete dead api.rs#126
phildenhoff merged 2 commits into
mainfrom
phil/cdl-5-chore-delete-dead-crateslibcalibresrcapirs

Conversation

@phildenhoff

Copy link
Copy Markdown
Member

crates/libcalibre/src/api.rs declared pub mod authors; pub mod book_files; pub mod books; for modules that no longer exist. It only ever compiled because lib.rs never declared mod api. Leftover from the pre-facade structure.

Verified nothing references mod api / api::, and cargo build -p libcalibre is clean after removal.

Fixes CDL-5

🤖 Generated with Claude Code

api.rs declared pub mod authors/book_files/books for modules that no
longer exist; it only compiled because lib.rs never declared mod api.
Leftover from the pre-facade structure. Nothing references it.

Fixes CDL-5

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 80.81%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@phildenhoff phildenhoff enabled auto-merge (squash) June 14, 2026 08:58
@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 80.81%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@phildenhoff phildenhoff merged commit 36f9782 into main Jun 14, 2026
7 checks passed
@phildenhoff phildenhoff deleted the phil/cdl-5-chore-delete-dead-crateslibcalibresrcapirs branch June 14, 2026 09:08
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