-
-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Problems running docs project locally #1307
Comments
Good point and thanks for reporting - I will fix that |
Fixed by #1306 |
For the time being @Jcparkyn
|
If you rebase the current |
@linkdotnet I think problems 4 and 5 from above are still present (running from the latest See this screenshot, where the TOC and search don't work, the xrefs are broken, and there was a SyntaxError from docfx.min.js: I also have nothing generated in |
Yeah I think that late issue is due to the template needs updating. |
Describe the bug
There seem to be a few issues with running the docs project locally (I'm putting them all in one issue because I assume most are related):
The docs readme says to run
docfx
fromdocs/
, but this gives an errorCannot find config file [...]\bUnit\docs\docfx.json
. Should this bedocfx site/docfx.json
or "run fromdocs/site/
"?I get a compilation error when running
docfx
:From
docs-deploy.yml
, it looks like I shouldn't even be runningdocfx
and instead usedotnet run Site.csproj
?Due to the above compilation error, the API spec doesn't get generated (with either of the above methods) unless I set
"allowCompilationErrors": true
indocfx.json
.docfx.min.js
doesn't get loaded due to "Cannot use import statement outside a module". This is fixed by settingtype="module"
on the relevant script tag.The deployed docs don't have correct formatting for block quotes (e.g. see "The current thread is not associated" at https://bunit.dev/docs/interaction/trigger-renders.html) - but this seems to be fixed when running locally.
Typo in "Verfiy docs samples" in
docs-deploy.yml
(doesn't cause any issues).Version info:
Additional context:
I think some or all of these are related to #1286
The text was updated successfully, but these errors were encountered: