Skip to content

app: Pyrefly#612

Merged
kevgo merged 4 commits intomainfrom
pyrefly
Mar 25, 2026
Merged

app: Pyrefly#612
kevgo merged 4 commits intomainfrom
pyrefly

Conversation

@kevgo
Copy link
Copy Markdown
Owner

@kevgo kevgo commented Mar 25, 2026

resolves #610

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

let ext = match platform.os {
Os::Linux | Os::MacOS => "tar.gz",
Os::Windows => "zip",
};
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Windows download URLs point to non-existent release assets

Medium Severity

The run_method generates Windows download URLs (pyrefly-windows-arm64.zip, pyrefly-windows-x86_64.zip), but Pyrefly's GitHub releases only include Linux and macOS binaries — no Windows assets exist. This will cause a download failure (404) at runtime for any Windows user attempting to install Pyrefly through this tool. The Windows tests validate these non-existent URLs, giving false confidence.

Additional Locations (1)
Fix in Cursor Fix in Web

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

This is not true. There are windows executables available for download.

@kevgo kevgo merged commit 70c83d9 into main Mar 25, 2026
3 checks passed
@kevgo kevgo deleted the pyrefly branch March 25, 2026 11:01
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.

app: pyrefly

1 participant