Re: docs: please use the :ref: directive instead of linking directly to documents

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks for fixing this, Alfredo.

On 08/14/2017 12:40 PM, Alfredo Deza wrote:
> While trying to update the dev section in the docs, found a lot of
> warnings for "unknown document" references.
> 
> This happens when using something like :doc:`/path/to/document`. The
> problem with that approach is that it isn't dynamic, if "document"
> moves then the link breaks.
> 
> A better approach is to rely on the :ref: directive which relies on
> labels. The caveat is that these labels must be unique, but as long as
> that is the case, one can reference it regardless of the location.
> 
> The Sphinx docs recommend using :ref:  (see [0]): "Using ref is
> advised over standard reStructuredText links to sections (like
> `Section title`_) because it works across files, when section headings
> are changed, and for all builders that support cross-references."
> 
> I'm going to go ahead and fix these:
> 
> /doc/rados/configuration/bluestore-config-ref.rst:184: WARNING:
> unknown document: /api/librados/#rados_set_alloc_hint
> /doc/rados/operations/health-checks.rst:164: WARNING: unknown
> document: /rados/operations/crush-map/#tunables
> /doc/rados/operations/health-checks.rst:176: WARNING: unknown
> document: /rados/operations/crush-map/#tunables
> /doc/rados/operations/health-checks.rst:330: WARNING: unknown
> document: placement-groups#Choosing-the-number-of-Placement-Groups
> /doc/rados/operations/health-checks.rst:351: WARNING: unknown
> document: placement-groups#Choosing-the-number-of-Placement-Groups
> /doc/rados/operations/health-checks.rst:404: WARNING: unknown
> document: pools.rst#associate-pool-to-application
> /doc/rados/operations/health-checks.rst:462: WARNING: unknown
> document: troubleshooting-pg#Unfound-objects
> /doc/release-notes.rst:440: WARNING: unknown document:
> /rados/operations/erasure-code/#erasure-code-profiles
> 
> Once that is fixed, I will enable "treat warnings as errors" in the
> docs build which will cause doc builds to fail, to prevent adding
> these unknowingly.
> 
> Thanks
> 
> [0] http://www.sphinx-doc.org/en/stable/markup/inline.html#role-ref
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


-- 
Dan Mick
Red Hat, Inc.
Ceph docs: http://ceph.com/docs
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux