-
-
Notifications
You must be signed in to change notification settings - Fork 103
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
doc: Use intra-doc-links instead of relative paths to html pages #1084
doc: Use intra-doc-links instead of relative paths to html pages #1084
Conversation
d135fed
to
9f6d507
Compare
@sophie-h Rebased and conflict automatically resolved (first commit was already applied), please review on gtk-rs :) |
I try. Great chance to understand the separated lgpl-docs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything else looks great. (Not an expert in this at all though.)
9f6d507
to
cb6d54f
Compare
cb6d54f
to
a3fe920
Compare
Using intra-doc-links saves quite a bit of code in gir, looks cleaner in the markdown files and resulting docs (`[src]` button mostly), and has the advantage of being checked by `rustdoc` for validity. That last point is not totally fair as most of these trait "implements" / "implementors" segments are generated inside macros, for which `rustdoc` does not seem to emit any warnings or errors currently.
a3fe920
to
70c312b
Compare
This supposedly also fixes broken links in CC @bilelmoussaoui :) |
Nice, thanks! |
Using intra-doc-links saves quite a bit of code in gir, looks cleaner in the markdown files and resulting docs (
[src]
button mostly), and has the advantage of being checked byrustdoc
for validity.That last point is not totally fair as most of these trait "implements" / "implementors" segments are generated inside macros, for which
rustdoc
does not seem to emit any warnings or errors currently.This has only been tested on GStreamer-rs, and only sampled at random because of aforementioned "no-warnings-in-macros" issue 😞