Skip to content
This repository has been archived by the owner on Jan 2, 2021. It is now read-only.

Generate doc file URL via LSP (to fix it for Windows) #721

Merged
merged 2 commits into from
Sep 2, 2020

Conversation

DunetsNM
Copy link
Contributor

@DunetsNM DunetsNM commented Aug 5, 2020

Part of the fix for haskell/haskell-language-server#247

Windows understands only file:/// but not file:// (at least it's true for VS Code)

@digitalasset-cla
Copy link

digitalasset-cla commented Aug 5, 2020

CLA assistant check
All committers have signed the CLA.

@DunetsNM
Copy link
Contributor Author

DunetsNM commented Aug 5, 2020

replaced "file://" hardcode with LSP normalizer, this fixes both three slashes for windows and replaces backslashes with forward slash (as mentioned in haskell/vscode-haskell/pull/270) and makes it work for Windows

Windows, VS Code + ghcide via Haskell (HLS) 1.0.1 works fine now:

image

@DunetsNM DunetsNM changed the title Use three slashes in doc file URLs (to fix it for Windows) Generate doc file URL via LSP (to fix it for Windows) Aug 5, 2020
@pepeiborra pepeiborra merged commit cf5df3c into haskell:master Sep 2, 2020
@pepeiborra
Copy link
Collaborator

Thanks you, and apologies for the long delay in merging your PR!

@DunetsNM DunetsNM deleted the fix-docs-url-for-windows branch September 4, 2020 12:52
pepeiborra pushed a commit to pepeiborra/ide that referenced this pull request Dec 29, 2020
…#721)

* use three slashes in doc file URLs to fix it for Windows

* generate normalized doc file URL using LSP
pepeiborra pushed a commit to pepeiborra/ide that referenced this pull request Dec 29, 2020
…#721)

* use three slashes in doc file URLs to fix it for Windows

* generate normalized doc file URL using LSP
pepeiborra pushed a commit to pepeiborra/ide that referenced this pull request Dec 29, 2020
…#721)

* use three slashes in doc file URLs to fix it for Windows

* generate normalized doc file URL using LSP
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants