Closed spdustin closed 1 week ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request modify the _get_indexer_from_json
method of the Prowlarr
class, specifically how movie_search_capabilities
and tv_search_capabilities
are assigned. The implementation shifts from generator expressions to list comprehensions for better evaluation of indexer capabilities. Additionally, a minor formatting adjustment was made in the _log_indexers
method by adding a newline at the end of the file.
File Path | Change Summary |
---|---|
src/program/services/scrapers/prowlarr.py | Modified _get_indexer_from_json : changed generator expression for capabilities to list comprehensions. Added newline in _log_indexers . |
In the code where bunnies hop,
A change was made, we won't stop!
Capabilities now flow with grace,
In Prowlarr's heart, they find their place.
With a newline added, neat and bright,
Our code now shines, a pure delight! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description:
The current
_get_indexer_from_json()
private method only stores the first character of an indexer'smovieSearchParams
/tvSearchParams
, resulting in Riven only providing the show/movie name to Prowlarr indexers even if they're able to search by year/season/episode.This commit fixes the issue by removing the 0-slice from the list comprehension.
PR info
Summary by CodeRabbit
Bug Fixes
Style