FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Use pathToFileURL to handle windows file path by t8g · Pull Request #475 · vercel/micro · GitHub

/ micro Public

Use pathToFileURL to handle windows file path - #475

Open
t8g wants to merge 1 commit into
vercel:mainfrom
t8g:pathtofileurl
Open

Use pathToFileURL to handle windows file path#475
t8g wants to merge 1 commit into
vercel:mainfrom
t8g:pathtofileurl

Conversation

t8g commented Jan 14, 2023

Copy link
Copy Markdown

Related to #474

Fix On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

leerob commented Jan 14, 2023

Copy link
Copy Markdown
Contributor

Could you add a test please?

t8g commented Jan 16, 2023

Copy link
Copy Markdown
Author

The current tests are already failing on windows.
And the PR make them pass.
But I don't know how to emulate the error on non windows plateform.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL