| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
FTR, I checked that there is no existing ref's to these in other parts of the docs (NEWS items, What's New, etc.) |
Sorry, something went wrong.
There was a problem hiding this comment.
Just to note, perhaps a little pedantic but the ref targets aren't completely syntactically and semantically redundant—their targets are the sections, while the :class: directive target is the class item, which can make both a syntactic and semantic difference in some contexts. If content outside the class directive, such as an introduction, examples, or other classes/objects is added to the section, then the class directive is not practically equivalent, especially so in practice if the content is added above the linked class, or in a different scope.
In this case, the only current examples of this are a versionadded directive above the Blob class, and some rather lengthy examples below the Row class (which besides being a little lengthy for the context, are somewhat confusing, since it references "the example given above", but is unclear which of the many examples this is—evidently the one in the document introduction). However, additional such material could be added at any point in the future, in which case the refs should presumably be re-added.
Whether they are worthwhile to keep, though, is up to you.
Some other more minor notes:
Sorry, something went wrong.
|
Nah, there's more important stuff to do with the sqlite3 docs than something like this. Churn for little value. I'm closing the PR. Thanks! |
Sorry, something went wrong.
|
Probably spent too much time writing that comment too myself 😅 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Each type can be ref'ed using the :class: directive; there is no reason to provide duplicate :ref:'s.