Skip to content
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

Misc fixes #898

Merged
merged 6 commits into from
Dec 3, 2018
Merged

Misc fixes #898

merged 6 commits into from
Dec 3, 2018

Conversation

fredrikekre
Copy link
Member

No description provided.

@fredrikekre
Copy link
Member Author

I still have this warning:

┌ Warning: 1 docstring potentially missing:
│ 
│     Documenter.Documenter
└ @ Documenter.DocChecks ~/dev/Documenter/src/DocChecks.jl:50

but that feels like a false alarm, because that is included here https://github.com/JuliaDocs/Documenter.jl/blob/master/docs/src/lib/public.md#L10 and shows up in the rendered docs.

@mortenpi
Copy link
Member

mortenpi commented Dec 2, 2018

I still have this warning:

┌ Warning: 1 docstring potentially missing:
│ 
│     Documenter.Documenter
└ @ Documenter.DocChecks ~/dev/Documenter/src/DocChecks.jl:50

but that feels like a false alarm, because that is included here https://github.com/JuliaDocs/Documenter.jl/blob/master/docs/src/lib/public.md#L10 and shows up in the rendered docs.

Yeah, we've been getting this for a long time. I have been meaning to debug this, but haven't gotten around to it. It seems like something about the Julia docstring internals changed subtly at some point.

@mortenpi mortenpi added this to the 0.21.0 milestone Dec 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants