Skip to content

refactor: rename NoRestartExplorer parameter to SkipExplorerRestart - #709

Merged
Raphire merged 1 commit into
masterfrom
rename-norestartexplorer
Jul 19, 2026
Merged

refactor: rename NoRestartExplorer parameter to SkipExplorerRestart#709
Raphire merged 1 commit into
masterfrom
rename-norestartexplorer

Conversation

@Raphire

@Raphire Raphire commented Jul 19, 2026

Copy link
Copy Markdown
Owner

The old NoRestartExplorer parameter is kept as an alias and will continue to work, but is considered deprecated

Summary by CodeRabbit

  • Renamed the Explorer restart option to -SkipExplorerRestart.
  • Preserved -NoRestartExplorer as a deprecated alias.
  • Updated restart handling, configuration import, GUI initialization, parameter tracking, and related tests to use the new name.

The old `NoRestartExplorer` parameter is kept as an alias and will
continue to work, but is considered deprecated
@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3150139b-e452-4676-9210-bfb3dd925264

📥 Commits

Reviewing files that changed from the base of the PR and between 9c033db and db25743.

📒 Files selected for processing (7)
  • Scripts/Features/Invoke-RestartExplorer.ps1
  • Scripts/GUI/Show-MainWindow.ps1
  • Scripts/Get.ps1
  • Scripts/Helpers/Import-ConfigToParams.ps1
  • Tests/Import-ConfigToParams.Tests.ps1
  • Tests/Resolve-UserProfilePath.Tests.ps1
  • Win11Debloat.ps1

📝 Walkthrough

Walkthrough

The Explorer restart control is renamed from NoRestartExplorer to SkipExplorerRestart across script parameters, configuration import, restart handling, GUI initialization, and tests. NoRestartExplorer remains supported as an alias.

Changes

Explorer restart parameter

Layer / File(s) Summary
Parameter contract and selection tracking
Win11Debloat.ps1, Scripts/Get.ps1
The primary switch parameter is renamed to SkipExplorerRestart, retains NoRestartExplorer as an alias, and is included in control-parameter selection tracking.
Skip propagation and validation
Scripts/Helpers/Import-ConfigToParams.ps1, Scripts/Features/Invoke-RestartExplorer.ps1, Scripts/GUI/Show-MainWindow.ps1, Tests/Import-ConfigToParams.Tests.ps1, Tests/Resolve-UserProfilePath.Tests.ps1
Configuration import, Explorer restart suppression, and GUI checkbox initialization use SkipExplorerRestart; tests cover the new parameter and legacy alias.
Estimated code review effort: 2 (Simple) ~10 minutes

Possibly related PRs

🚥 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 describes the parameter rename and alias preservation in the PR.
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.
Powershell Docstrings ✅ Passed Touched functions keep comment-based help, and their descriptions still fit the renamed SkipExplorerRestart flow.

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.

@Raphire
Raphire merged commit a9c1736 into master Jul 19, 2026
2 checks passed
@Raphire
Raphire deleted the rename-norestartexplorer branch July 19, 2026 20:58
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