Skip to content

fix(config): correct scoop_repo help URL#6642

Closed
B67687 wants to merge 1 commit intoScoopInstaller:masterfrom
B67687:fix/scoop-config-scoop-repo-help
Closed

fix(config): correct scoop_repo help URL#6642
B67687 wants to merge 1 commit intoScoopInstaller:masterfrom
B67687:fix/scoop-config-scoop-repo-help

Conversation

@B67687
Copy link
Copy Markdown

@B67687 B67687 commented Apr 24, 2026

Summary

  • update the documented scoop_repo default URL in scoop config --help
  • make the help text match the actual default config value

Closes #6633.

Summary by CodeRabbit

  • Documentation
    • Updated configuration help text to reflect HTTPS repository URL.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e8848664-9726-43db-a6bd-187d082d0fa2

📥 Commits

Reviewing files that changed from the base of the PR and between b588a06 and b9b1309.

📒 Files selected for processing (1)
  • libexec/scoop-config.ps1

Walkthrough

This PR updates the inline help documentation for the scoop_repo configuration setting, changing the protocol reference from HTTP to HTTPS to align with the actual configuration value and resolve a documentation inconsistency.

Changes

Cohort / File(s) Summary
Help Documentation Update
libexec/scoop-config.ps1
Updated the help text for the scoop_repo configuration setting, changing the protocol reference from http to https to match the actual configuration value.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny fix, but oh so keen,
The protocol now clean and keen,
From HTTP to HTTPS bright,
The docs and config match just right!
Hop-hop, the help text shines anew! 🌟

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: correcting the scoop_repo help URL from http to https to match the actual configuration default.
Linked Issues check ✅ Passed The PR directly addresses issue #6633 by correcting the documented scoop_repo URL in help text from http to https, matching the actual configuration value.
Out of Scope Changes check ✅ Passed The PR contains only the necessary change to fix the documented URL discrepancy; no extraneous modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@B67687 B67687 closed this Apr 29, 2026
@B67687
Copy link
Copy Markdown
Author

B67687 commented May 1, 2026

This PR was closed because it targeted master instead of develop. The Scoop project requires all PRs to target the develop branch. Please see the replacement PR #6649 which targets develop and is currently open.

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.

[Bug] scoop-config scoop_repo --help Match Config

1 participant