Skip to content

Conversation

@404Wolf
Copy link
Collaborator

@404Wolf 404Wolf commented Jul 5, 2025

Support windows style paths :(. Achieved via asPosixPath and abstracting all API methods to utilities in sdk.ts (which I have mixed feelings about, but feels like it's for the better)
.

@404Wolf 404Wolf force-pushed the fix-windows-paths branch 3 times, most recently from b8134ee to 6ed6751 Compare July 5, 2025 18:17
@404Wolf 404Wolf force-pushed the fix-windows-paths branch 2 times, most recently from d39c3dd to fbf1c91 Compare July 5, 2025 18:26
@404Wolf 404Wolf force-pushed the fix-windows-paths branch from fbf1c91 to 914d1d7 Compare July 5, 2025 18:34
@404Wolf 404Wolf changed the title Fix windows paths Add Windows Support Jul 5, 2025
@404Wolf 404Wolf force-pushed the fix-windows-paths branch 2 times, most recently from 7ef56ef to 7ef610e Compare July 5, 2025 21:06
@404Wolf 404Wolf force-pushed the fix-windows-paths branch from 7ef610e to 7d3971c Compare July 5, 2025 21:08
@404Wolf 404Wolf requested a review from maxmcd July 5, 2025 21:27
@404Wolf 404Wolf force-pushed the fix-windows-paths branch 2 times, most recently from f8234ec to 192e9a1 Compare July 6, 2025 01:04
@404Wolf 404Wolf force-pushed the fix-windows-paths branch from 192e9a1 to 7565fd6 Compare July 6, 2025 01:05
@404Wolf
Copy link
Collaborator Author

404Wolf commented Jul 25, 2025

@maxmcd can I merge this?

@404Wolf
Copy link
Collaborator Author

404Wolf commented Sep 10, 2025

@maxmcd can I get a review on this? This branch is continuing to fall behind and vt is still broken on windows.

Copy link
Member

@tmcw tmcw left a comment

Choose a reason for hiding this comment

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

Go for it!

@404Wolf 404Wolf force-pushed the fix-windows-paths branch 3 times, most recently from b3fa5fa to efe0fbe Compare September 10, 2025 20:22
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.

3 participants