Skip to content

Fix: Address PR #401 review suggestions - #402

Merged
masonfox merged 3 commits into
developfrom
fix/pr-401-review-suggestions
Mar 19, 2026
Merged

Fix: Address PR #401 review suggestions#402
masonfox merged 3 commits into
developfrom
fix/pr-401-review-suggestions

fix: address GitHub Copilot review feedback

0844ad1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 19, 2026 in 0s

78.62% (-0.03%) compared to 54d00f9

View this Pull Request on Codecov

78.62% (-0.03%) compared to 54d00f9

Details

Codecov Report

❌ Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/query-keys.ts 0.00% 2 Missing ⚠️
hooks/useShelfBooks.ts 87.50% 0 Missing and 1 partial ⚠️

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #402      +/-   ##
===========================================
- Coverage    78.64%   78.62%   -0.03%     
===========================================
  Files          167      167              
  Lines         7538     7540       +2     
  Branches      1846     1846              
===========================================
  Hits          5928     5928              
- Misses        1125     1127       +2     
  Partials       485      485              
Files with missing lines Coverage Δ
hooks/useLibraryData.ts 78.84% <ø> (ø)
lib/hooks/usePageTitle.ts 70.58% <ø> (ø)
hooks/useShelfBooks.ts 96.36% <87.50%> (ø)
lib/query-keys.ts 94.11% <0.00%> (-5.89%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.