Skip to content

add multilist examples - #34

Merged
ryanpag3 merged 2 commits into
mainfrom
add-additional-list-type-support
Nov 12, 2025
Merged

add multilist examples#34
ryanpag3 merged 2 commits into
mainfrom
add-additional-list-type-support

Conversation

@ryanpag3

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings November 12, 2025 01:50
@ryanpag3
ryanpag3 merged commit 54b9bf2 into main Nov 12, 2025
1 check passed
@github-actions

Copy link
Copy Markdown

Coverage after merging add-additional-list-type-support into main will be

97.76%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   index.ts92%50%100%94.44%33–34
src/api
   radarr.ts96%88.24%100%97.98%114, 148, 183–185, 193
src/scraper
   index.ts89.47%84.21%100%91.18%53, 64, 64–66, 85
   list.ts98.41%91.67%100%100%64
   movie.ts100%100%100%100%
src/util
   env.ts100%100%100%100%
   logger.ts100%100%100%100%

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive examples and documentation for running multiple lettarrboxd instances to monitor multiple Letterboxd lists simultaneously.

  • Adds multi-list deployment examples in both docker-compose and Docker CLI formats
  • Documents best practices for running multiple instances with separate data directories, tags, and configurations
  • Provides commented-out service definitions in docker-compose.yaml for easy activation

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docker-compose.yaml Adds single instance comment clarification and three commented multi-list service examples with different configurations
README.md Adds new "Watching Multiple Lists" section with Docker Compose and CLI examples, plus best practices guide

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
5. **Quality Profiles**: Each instance can use different quality profiles based on content type:
```yaml
- RADARR_QUALITY_PROFILE=HD-1080p # Standard content
- RADARR_QUALITY_PROFILE=Ultra HD # Premium content

Copilot AI Nov 12, 2025

Copy link

Choose a reason for hiding this comment

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

[nitpick] The spacing before the comment is inconsistent with the other example. In the Docker Compose multi-list example (line 199), there are multiple spaces before the comment, while in the docker-compose.yaml file (line 67), there is only one space before the comment. For consistency, consider using single space before inline comments throughout.

Suggested change
- RADARR_QUALITY_PROFILE=Ultra HD # Premium content
- RADARR_QUALITY_PROFILE=Ultra HD # Premium content

Copilot uses AI. Check for mistakes.
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