Skip to content

Feature: Add list unread books command#168

Closed
ngvalentino wants to merge 2 commits into
github:mainfrom
ngvalentino:feature/add-list-unread
Closed

Feature: Add list unread books command#168
ngvalentino wants to merge 2 commits into
github:mainfrom
ngvalentino:feature/add-list-unread

Conversation

@ngvalentino

Copy link
Copy Markdown

Adds BookCollection.get_unread_books(), CLI handlers for 'unread' and 'list-unread', updated help text, and comprehensive unit & CLI tests.

ngvalentino and others added 2 commits June 22, 2026 08:32
- Add search() method for case-insensitive title/author searching
- Add review system: add_review(), get_reviews(), average_rating()
- Add handle_search() and handle_review() commands
- Refactor command dispatch to use dictionary mapping
- Add comprehensive type hints to book_app.py and books.py
- Enhance find_by_author() to support substring matching
- Add tests for search and review features
- Update README with new search command

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…read_books()\n- Add CLI handlers for 'unread' and 'list-unread'\n- Update help text\n- Add unit and CLI tests\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@DanWahlin

Copy link
Copy Markdown
Collaborator

Thanks for the PR @ngvalentino. The code in the course is designed to generate specific output for Copilot CLI reviews and other actions performed throughout the course so we're not looking to change it right now or add additional features (you'll find information about that in the root readme's contributions section). If we do make changes in the future we'll refer back to this PR though. Thank you.

@DanWahlin DanWahlin closed this Jun 24, 2026
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.

2 participants