Update Windows installation instructions in README#347
Merged
junaid2005p merged 1 commit intomainfrom Apr 11, 2026
Merged
Conversation
| | **Arch Linux (AUR)** | `yay -S surge` | Managed via AUR. | | ||
| | **macOS / Linux (Homebrew)** | `brew install SurgeDM/tap/surge` | Recommended for Mac/Linux users. | | ||
| | **Windows** | `winget install SurgeDM.surge`<br />or<br />`scoop install surge` | Recommended for Windows users. | | ||
| | **Windows** | `winget install surge-downloader.surge`<br />or<br />`scoop install surge` | Recommended for Windows users. | |
Contributor
There was a problem hiding this comment.
Other
SurgeDM references may also be outdated
The brew tap (brew install SurgeDM/tap/surge) and Go install path (go install github.com/SurgeDM/Surge@latest) still use the old SurgeDM namespace. The upstream GitHub repo now lives under surge-downloader, and the official docs show brew install surge-downloader/tap/surge. If the Homebrew tap has been renamed, the brew command would silently fail for users. Consider updating these in the same pass.
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 67
Comment:
**Other `SurgeDM` references may also be outdated**
The brew tap (`brew install SurgeDM/tap/surge`) and Go install path (`go install github.com/SurgeDM/Surge@latest`) still use the old `SurgeDM` namespace. The upstream GitHub repo now lives under `surge-downloader`, and the official docs show `brew install surge-downloader/tap/surge`. If the Homebrew tap has been renamed, the brew command would silently fail for users. Consider updating these in the same pass.
How can I resolve this? If you propose a fix, please make it concise.
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.
Greptile Summary
This PR updates the Windows
wingetinstall command in the installation table fromwinget install SurgeDM.surgetowinget install surge-downloader.surge. The new package identifier is confirmed correct per the official winget package registry (winstall.app) and the project's own published documentation.brew install SurgeDM/tap/surgecommand andgo install github.com/SurgeDM/Surge@lateststill reference the oldSurgeDMorganization, which may also be outdated given the project appears to have migrated to thesurge-downloadernamespace. These are not introduced by this PR but may be worth a follow-up.Confidence Score: 5/5
Safe to merge — the winget package ID change is verified correct and no logic or code is affected.
Single-line documentation fix with the new package identifier confirmed against the official winget registry and project docs. The only finding is a P2 suggestion to also update other lingering
SurgeDMreferences in the same file.README.md — other
SurgeDMreferences (brew tap, Go install URL) may warrant a follow-up update.Important Files Changed
SurgeDM.surgetosurge-downloader.surge; change is accurate per the official winget registry.Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[Windows User] --> B{Choose installer} B --> C[winget] B --> D[scoop] C --> E["winget install surge-downloader.surge\n(updated from SurgeDM.surge)"] D --> F["scoop install surge\n(unchanged)"] E --> G[Surge installed ✓] F --> GPrompt To Fix All With AI
Reviews (1): Last reviewed commit: "Update Windows installation instructions..." | Re-trigger Greptile