Conversation
Contributor
审查者指南(在小型 PR 上折叠)审查者指南通过在目标路径为目录时设置 UseShellExecute,确保在启动进程以打开路径时,目录始终通过 shell 打开,同时保留现有的管理员权限提升行为。 目录 shell 执行的进程启动流程图flowchart TD
A[startProcess path is passed] --> B[Create ProcessStartInfo psi]
B --> C{isAdmin}
C -->|true| D[psi.UseShellExecute = true]
D --> E[psi.Verb = runas]
C -->|false| F[no admin changes]
E --> G{Directory.Exists path}
F --> G
G -->|true| H[psi.UseShellExecute = true]
G -->|false| I[leave psi.UseShellExecute as is]
H --> J[Process.Start psi]
I --> J
文件级变更
与关联 Issue 的对照评估
可能相关的 Issue
技巧与命令与 Sourcery 交互
自定义你的体验访问你的 仪表盘 以:
获取帮助Original review guide in EnglishReviewer's guide (collapsed on small PRs)Reviewer's GuideEnsures that when starting a process to open a path, directories are always opened via the shell by setting UseShellExecute when the target path is a directory, in addition to the existing admin-elevation behavior. Flow diagram for process start with directory shell executionflowchart TD
A[startProcess path is passed] --> B[Create ProcessStartInfo psi]
B --> C{isAdmin}
C -->|true| D[psi.UseShellExecute = true]
D --> E[psi.Verb = runas]
C -->|false| F[no admin changes]
E --> G{Directory.Exists path}
F --> G
G -->|true| H[psi.UseShellExecute = true]
G -->|false| I[leave psi.UseShellExecute as is]
H --> J[Process.Start psi]
I --> J
File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - 我已经审阅了你的更改,一切看起来很棒!
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进审查质量。
Original comment in English
Hey - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Pigeon0v0
approved these changes
Jul 12, 2026
Hill23333
approved these changes
Jul 13, 2026
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.
close #3379
Summary by Sourcery
Bug Fixes:
Original summary in English
Summary by Sourcery
Bug Fixes: