Update index.md - #2749
Draft
platnub wants to merge 2 commits into
Draft
Conversation
Contributor
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds a documentation note instructing users to verify the python3 installation path in Docker when configuring SABnzbd scripts, to prevent script execution failures. Flow diagram for updated SABnzbd script setup checklistflowchart TD
A[Create scripts folder in SABnzbd root] --> B[Set scripts folder in SABnzbd settings]
B --> C[Ensure script has execute permissions]
C --> D[Check python3 path with `which python3` and update script shebang]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
|
Deploying with ⚡ Cloudflare Pages
|
Contributor
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- Consider clarifying that the python3 path should be used in the script’s shebang line (e.g.,
#!/usr/bin/env python3or a concrete path) so users know exactly where to apply the result ofwhich python3. - It may be helpful to note that
which python3is intended for Unix-like environments and briefly mention an equivalent approach for Windows users, or explicitly scope the instruction to Linux/macOS.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Consider clarifying that the python3 path should be used in the script’s shebang line (e.g., `#!/usr/bin/env python3` or a concrete path) so users know exactly where to apply the result of `which python3`.
- It may be helpful to note that `which python3` is intended for Unix-like environments and briefly mention an equivalent approach for Windows users, or explicitly scope the instruction to Linux/macOS.
## Individual Comments
### Comment 1
<location path="docs/Downloaders/SABnzbd/scripts/index.md" line_range="14" />
<code_context>
- You've created folder called `scripts` in the root directory of SABnzbd
- You've set the `scripts` folder inside the SABnzbd settings under `Folder > User Folders > Scripts Folder`. ([More Infos](https://sabnzbd.org/wiki/configuration/4.3/folders))
- Your script got sufficient rights to execute. ([More Infos](https://sabnzbd.org/wiki/configuration/4.5/scripts/post-processing-scripts))
+- You've checked the installation location for python3 using `which python3` to replace the path at the start of the script.
## Clean
</code_context>
<issue_to_address>
**nitpick (typo):** Consider capitalizing and spacing "Python 3" as a proper name.
In this bullet, use “Python 3” in the prose, while keeping the command `which python3` unchanged in backticks.
```suggestion
- You've checked the installation location for Python 3 using `which python3` to replace the path at the start of the script.
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Contributor
|
Deploying with ⚡ Cloudflare Pages
|
Contributor
|
Thanks for this PR. I approved your suggestion, but you still need to fix a few things before it's approved. |
Contributor
bakerboy448
marked this pull request as draft
June 21, 2026 03:53
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added instruction to check python3 path in Docker.
Pull Request
Purpose
Scripts don't run if python3 path is incorrect.
Approach
This informs users that the python3 path is important for the scripts to run successfully and a basic step how to fix it.
Open Questions and Pre-Merge TODOs
Requirements
Summary by Sourcery
Documentation: