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

Resolve doc warning with global enums #896

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

fpdotmonkey
Copy link
Contributor

If you run cargo doc, as apposed to cargo doc -p godot --no-libs as in the CI, you get hit with a warning "warning: redundant explicit link target" on the docs in this commit. I'm not sure why these options together supress the warning (neither individually does). But this reduces noise in developer consoles.

If you run `cargo doc`, as apposed to `cargo doc -p godot --no-libs` as
in the CI, you get hit with a warning "warning: redundant explicit link
target" on the docs in this commit.  I'm not sure why these options
together supress the warning (neither individually does).  But this
reduces noise in developer consoles.
@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-896

@Bromeon Bromeon added quality-of-life No new functionality, but improves ergonomics/internals documentation Improvements or additions to documentation labels Sep 14, 2024
@Bromeon
Copy link
Member

Bromeon commented Sep 14, 2024

Thank you! I remember this issue, cool that you found such a straightforward way to address it! 👍

@Bromeon Bromeon added this pull request to the merge queue Sep 14, 2024
Merged via the queue into godot-rust:master with commit f9e7391 Sep 14, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation quality-of-life No new functionality, but improves ergonomics/internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants