Skip to content

chore: Remove explicit VerbosePreference setting, default to SilentlyContinue#76

Merged
chawyehsu merged 2 commits into
ScoopInstaller:mainfrom
z-Fng:set-verbose-preference
May 13, 2026
Merged

chore: Remove explicit VerbosePreference setting, default to SilentlyContinue#76
chawyehsu merged 2 commits into
ScoopInstaller:mainfrom
z-Fng:set-verbose-preference

Conversation

@z-Fng
Copy link
Copy Markdown
Member

@z-Fng z-Fng commented May 5, 2026

Motivation and Context

Most Verbose outputs are unnecessary at the moment. Setting VerbosePreference to SilentlyContinue can suppress these outputs.

Relates to:

Summary by CodeRabbit

  • Chores
    • Reduced verbose output during action execution to produce cleaner logs and a smoother user experience.
    • Maintained error handling behavior while removing the explicit verbose setting to rely on default or external logging controls.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 5, 2026

Walkthrough

Removed the single assignment that set $Global:VerbosePreference in action.ps1; the script no longer explicitly configures the global verbose preference.

Changes

Global Verbose Preference

Layer / File(s) Summary
Remove global verbose assignment
action.ps1
Deleted the line assigning $Global:VerbosePreference = 'Continue'; no other changes to global preference handling.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I’m a rabbit in the shell so spry,
I hopped through lines and said goodbye.
The noisy logs I now unbind,
A quieter trail is what I find.
🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately reflects the main change: removing the explicit VerbosePreference setting to allow PowerShell to default to SilentlyContinue.

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

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

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

@z-Fng
Copy link
Copy Markdown
Member Author

z-Fng commented May 12, 2026

It's been nearly six months since PR #67 was completed. I've simplified the changes and resubmitted a new pull request. I was wondering if it would be possible to merge this pull request? Or are there any thoughs on whether this pull request should be merged?

Comment thread action.ps1 Outdated
…stead

Co-authored-by: Chawye Hsu <chawyehsu@hotmail.com>
@z-Fng z-Fng changed the title chore: Change default VerbosePreference to SilentlyContinue chore: Remove explicit VerbosePreference setting, default to SilentlyContinue May 13, 2026
@chawyehsu chawyehsu merged commit 63f20ef into ScoopInstaller:main May 13, 2026
2 checks passed
@z-Fng z-Fng deleted the set-verbose-preference branch May 13, 2026 21:07
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