Comments (15)
Hello @felix-alonso, I just tried these steps on Windows and could view the API docs there.
So, maybe this wasn't designed to work on anything else than Windows. The workflows seem to support this theory.
I can help you with the fsharp-core-docs
repository once the fix is available in fsdocs.
Should change all of the instances of docs.microsoft.com/ to learn.microsoft.com/?
Yes, I believe so, if possible maybe refactor things a bit so that we use "learn.microsoft.com/"
as a single constant.
Thanks again for looking into this!
from fsharp.formatting.
Does this mean I get to put FSharp core contributor on my resume? 🤣
Thank you!
from fsharp.formatting.
it looks like this has (partially) already been fixed in FSharp.Formatting - the URL generation uses docs.microsoft.com: https://github.dev/fsprojects/FSharp.Formatting/blob/a9ecebc3db319f2a3d07f841755ffc61bbf936df/src/FSharp.Formatting.ApiDocs/GenerateModel.fs#L950.
However, this should be updated to use learn.microsoft.com
instead as a root.
Separately, the https://github.com/fsharp/fsharp-core-docs repo should update to a more recent FSharp.Formatting and republish the docs to get the fix.
from fsharp.formatting.
Hi Felix, are you interested in contributing here?
from fsharp.formatting.
Possibly! I'm just getting started with F#. How could I help?
When I clone and build the repo, I'm getting an MS Build exception.
I'm running:
OS:
Linux 5.10.0-20-amd64 #1 SMP Debian 5.10.158-2 (2022-12-13) x86_64 GNU/Linux
.NET: v 7.0.102
IDE:
Version: 1.75.1
Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:35:30.018Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: 10.2.154.23-electron.0
OS: Linux x64 5.10.0-20-amd64
Sandboxed: No
Ionide.Ionide-fsharp: v7.4.2
Am I doing anything obviously wrong?
from fsharp.formatting.
I think I've gotten past the build issue. I can run the documentation locally. I'm just not sure how I can verify that I've fixed the external documentation link.
I see that this Classic XML Doc Comments
contains an external link, but it uses the other URL format. When I click on the link it redirects to the new URL format
Current: https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/xmldoc/
Redirect: https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/xmldoc/
When I look through the project it seems like there are half a dozen other references to that URL.
Should change all of the instances of docs.microsoft.com/
to learn.microsoft.com/
?
from fsharp.formatting.
Thanks for the help!
I'm just not sure how I can verify that I've fixed the external documentation link.
The fsharp core docs are here https://github.com/fsharp/fsharp-core-docs
You’d want to build that repo’s docs using the fsdocs tool built with your PR. You can build those docs following the Build Steps on the readme.
The key thing would be to clone the repos as described in the fsharp-core-docs build steps. Then make your proposed changes in the FSharp.Formatting repo and build fsdocs with your proposed changes. Then do
(from 'fsharp-core-docs')
FSharp.Formatting\src\FSharp.Formatting.CommandTool\bin\Release\netcoreapp3.1\fsdocs.exe watch --sourcefolder fsharp
It’ll pop up a browser with the fsharp docs and you can test the URLs.
from fsharp.formatting.
I've been able to mostly follow the build steps. I had to run the following to spin up the website:
(start in 'fsharp-core-docs')
dotnet restore FSharp.Core
(make 'fsharp-core-docs/fsharp' and 'fsharp-core-docs/FSharp.Formatting' )
git clone https://github.com/dotnet/fsharp --depth 1 -b main
git clone https://github.com/fsprojects/FSharp.Formatting --depth 1
(build 'fsharp-core-docs/fsharp'; didn't take noVisualStudio flag)
pushd fsharp
./build.sh
popd
(build 'fsharp-core-docs/FSharp.Formatting'; didn't take build flag)
pushd FSharp.Formatting
./build.sh
popd
(from 'fsharp-core-docs')
FSharp.Formatting/src/fsdocs-tool/bin/Release/net7.0/fsdocs watch --sourcefolder fsharp
However, it doesn't seem to pick up any modules in fsharp project. See the docs below:
Similarly, clicking on any of the documentation links it just complains that the connection was reset:
Any clue what might be causing this?
from fsharp.formatting.
I just found out that the links to System.String and System.Text.RegularExpressions are actually coming from:
So, this would need a fix in dotnet/fsharp
as well.
from fsharp.formatting.
I've marked this PR as ready for review. Thank you for your guidance in this process!
from fsharp.formatting.
I just found out that the links to System.String and System.Text.RegularExpressions are actually coming from:
So, this would need a fix in
dotnet/fsharp
as well.
Looks like these are the correct links:
- System.String
- Before:
http://msdn2.microsoft.com/en-us/library/system.string.aspx
- After:
https://learn.microsoft.com/en-us/dotnet/api/system.string
- Before:
- System.Text.RegularExpressions
- Before:
http://msdn2.microsoft.com/library/system.text.regularexpressions.aspx
- After:
https://learn.microsoft.com/en-us/dotnet/api/system.text.regularexpressions
- Before:
So the change is something like s|msdn2.microsoft.com/en-us/library|learn.microsoft.com/en-us/dotnet/api|g
and removing the .aspx
.
from fsharp.formatting.
@felix-alonso thanks! I've updated all the links at the compiler side: dotnet/fsharp#14774
Once, that is merged we can try and update https://github.com/fsharp/fsharp-core-docs
from fsharp.formatting.
@felix-alonso I believe this problem is resolved now. Please confirm.
from fsharp.formatting.
It's looking good! Thank you so much!
from fsharp.formatting.
Thanks for confirming.
from fsharp.formatting.
Related Issues (20)
- Add how-to for the most typical scenario HOT 2
- List users of this tool HOT 1
- CSharp library not used for generating API docs HOT 7
- Function signature tooltips include attributes for arguments that make the tooltip unreadable HOT 5
- Directory.Build.props is ignored if there are no projects or solutions HOT 3
- SEO-optimization for new theme HOT 4
- Suggestion: init command
- Dark mode borders: too much contrast?
- Namespace description overflows content box
- Output font size on mobile safari is too big HOT 6
- Using testcontainers in a fsx script gives wrong output HOT 7
- Create a keyword/glossary site map
- add .dib notebook support HOT 6
- --watch expectation HOT 1
- Suggestion: auto-expansion of `<details>…</details>` for direct links HOT 12
- Can FSharp.Formatting support documenting programs? HOT 1
- Enable `watch` server from arbitrary host HOT 3
- Is it possible to generate an HTML AST from Markdown? HOT 1
- Nesting Document Categories HOT 1
- Include images in documentation in generated content HOT 10
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from fsharp.formatting.