-
Notifications
You must be signed in to change notification settings - Fork 254
Improve docstrings and doc #360
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
Comments
I'ld suggest putting this on the backburner until #310. |
I noticed that one reason for splitting this package is that it should help fixing (future) dep warning. Is https://github.com/apps/femtocleaner installed ? |
Helps JuliaCollections#360 to find missing docstring
Helps #360 to find missing docstring
When trying to build doc, we get:
Maybe @mortenpi @fredrikekre or @KristofferC who are active contributor of Documenter.jl can tell us why we are facing this |
As far as I can tell, DataStructures does not define |
Thanks @mortenpi |
Doc should explain:
|
https://travis-ci.org/JuliaCollections/DataStructures.jl/jobs/413204515#L836 shows 89 potentially missing docstrings https://travis-ci.org/JuliaCollections/DataStructures.jl/jobs/413204515#L820 shows expanding markdown templates that are failing because no docs is found |
using |
Anyone here have ever used https://juliadocs.github.io/DocStringExtensions.jl/latest/ and |
Hello,
Doc need now to be improved
Example:
http://juliacollections.github.io/DataStructures.jl/latest/deque.html
(but it also apply to most doc page)
Instead of a usage paragraph with raw Julia code we should use docstrings and
@docs
Example of docstring for
isempty
https://github.com/JuliaLang/julia/blob/edeb002a13afdce31dac55706433ebb6d5cedd79/base/essentials.jl#L727
Writing doc this way will help to have a doc which really reflect current code (and avoid issues with documentation of constructors such as exposed in #340 )
Maybe a new release should be tagged before tackling this issue because it will be easier to compare latest version (with @docs) to previous (current) version with raw Julia code in doc.
Kind regards
The text was updated successfully, but these errors were encountered: