Skip to content

fix(config): correct scoop_repo help URL#6649

Open
B67687 wants to merge 1 commit into
ScoopInstaller:developfrom
B67687:fix/scoop-config-scoop-repo-help
Open

fix(config): correct scoop_repo help URL#6649
B67687 wants to merge 1 commit into
ScoopInstaller:developfrom
B67687:fix/scoop-config-scoop-repo-help

Conversation

@B67687
Copy link
Copy Markdown

@B67687 B67687 commented Apr 29, 2026

Summary

  • Update the documented scoop_repo default URL in scoop config --help to use HTTPS instead of HTTP
  • Correct typo: containining to containing
  • Make the help text match the actual default config value

Closes #6633.

Motivation and Context

The scoop config --help text showed an outdated HTTP URL for the default scoop_repo, which did not match the actual default value. This is a help-text-only fix with no runtime behavior changes.

How Has This Been Tested?

  • Ran scoop config --help and confirmed the output now shows the correct HTTPS URL
  • Confirmed the typo is corrected and the description is clearer

Checks

  • I have read the Contributing Guide.
  • I have ensured that I am targeting the develop branch.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly. (Not applicable: help-text only)
  • I have added an entry in the CHANGELOG. (Not applicable: help-text only)

Summary by CodeRabbit

Release Notes

  • Bug Fixes

    • Fixed typo in configuration help text.
  • Documentation

    • Updated repository configuration URL to use HTTPS protocol.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

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: 2978ce68-e41d-4505-9f80-9ef522b7d085

📥 Commits

Reviewing files that changed from the base of the PR and between eee819c and aac2918.

📒 Files selected for processing (1)
  • libexec/scoop-config.ps1
✅ Files skipped from review due to trivial changes (1)
  • libexec/scoop-config.ps1

Walkthrough

Updated help/comment text in libexec/scoop-config.ps1: changed the scoop_repo example URL to https:// and fixed a typo in the description. No executable logic or public API was modified.

Changes

Configuration Documentation

Layer / File(s) Summary
Update scoop_repo help text
libexec/scoop-config.ps1
Changed the example scoop_repo URL from http://... to https://... and corrected the typo "containining" → "containing". No behavioral changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

I'm a rabbit in the config glade,
I hopped from http to https today.
A tiny typo brushed away with care,
Help text now tidy, clean, and fair.
🐇🔒

🚥 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 PR title clearly and concisely describes the main change: correcting the scoop_repo help URL from HTTP to HTTPS and fixing the typo.
Linked Issues check ✅ Passed The PR directly addresses all requirements from issue #6633: updates the documented scoop_repo URL to HTTPS, corrects the typo, and ensures help text matches the actual configuration.
Out of Scope Changes check ✅ Passed All changes in the PR are directly related to the linked issue #6633 objectives: only the help text URL and typo were modified with no unrelated alterations.
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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@libexec/scoop-config.ps1`:
- Around line 36-39: Fix the typo in the help text comment: change the word
"containining" to "containing" in the comment line "#       Git repository
containining scoop source code." so the help text reads "#       Git repository
containing scoop source code." This keeps the existing surrounding comments and
aligns the help text with the URL change already made.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0c5b1f99-ffbc-487e-9000-b51835d6e882

📥 Commits

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

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

Comment thread libexec/scoop-config.ps1
@B67687 B67687 force-pushed the fix/scoop-config-scoop-repo-help branch from 2739836 to eee819c Compare April 29, 2026 09:07
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

- Change http:// to https:// for better security
- Fix typo in help text (containining → containing)
@B67687 B67687 force-pushed the fix/scoop-config-scoop-repo-help branch from f707a31 to aac2918 Compare May 22, 2026 01:57
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.

1 participant