-
Notifications
You must be signed in to change notification settings - Fork 22
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
adjust docs/make.jl
to changes in Documenter.jl
#945
adjust docs/make.jl
to changes in Documenter.jl
#945
Conversation
Codecov Report
@@ Coverage Diff @@
## master #945 +/- ##
=======================================
Coverage 75.70% 75.70%
=======================================
Files 51 51
Lines 4169 4169
=======================================
Hits 3156 3156
Misses 1013 1013 |
You can also add Alternatively one can set |
Now we can *almost* process the documentation with Documenter.jl 1.0. - The creation of hyperlinks to the GAP documentation works again. - The errors because of docstrings that are not referenced in `docs/src/*.md` are gone because the docstrings are now referenced. (I do not like this at all. Why is it an error not to include all available docstrings?) - There are still errors because three explicit cross-references to the JuliaInterface manual (in the directory `assets/html/JuliaInterface`) are not accepted because they are local links outside the build directory.
0bf3bff
to
e336b58
Compare
Oops, I kinda forgot about this PR. Everything looks good, tests pass -- is there any reason not to merge it? |
Oh wait, now I understand: the three errors you are referring to are for when one switches to actually using Documenter.jl 1.x -- which this PR does not, it just prepares for this. |
Now we can almost process the documentation with Documenter.jl 1.0.
docs/src/*.md
are gone because the docstrings are now referenced. (I do not like this at all. Why is it an error not to include all available docstrings?)assets/html/JuliaInterface
) are not accepted because they are local links outside the build directory.