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

Use link text in docs to omit Self:: prefix in rendered docs #561

Merged
merged 1 commit into from
Jan 24, 2022

Conversation

MarijnS95
Copy link
Collaborator

@MarijnS95 MarijnS95 commented Jan 24, 2022

Suggestion from #559.

Note that Self:: in _len() function docs hasn't been replaced (yet?) as the longer function names make this format rather verbose. However, we should probably prefer docs readability over code readability?

@Ralith
Copy link
Collaborator

Ralith commented Jan 24, 2022

However, we should probably prefer docs readability over code readability?

That's my general preference, subject to developer patience.

@MarijnS95 MarijnS95 changed the title prelude: Use link text in ColorComponentFlags docs to omit Self:: Use link text in docs to omit Self:: prefix in rendered docs Jan 24, 2022
@MarijnS95
Copy link
Collaborator Author

@Ralith Consider it regexed :)

Copy link
Collaborator

@Ralith Ralith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@@ -264,10 +264,10 @@ impl Entry {
&self.entry_fn_1_1
}

#[deprecated = "This function is unavailable and therefore panics on Vulkan 1.0, please use `try_enumerate_instance_version` instead"]
#[deprecated = "This function is unavailable and therefore panics on Vulkan 1.0, please use `try_enumerate_instance_version()` instead"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought (intradoc) links were also possible in deprecation blocks (similar to other markdown), but nope :(

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aw, that seems like a really useful place for them too. Feature request upstream?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heh feature requests have to go to the "internals" forum, I'll see what I can do.

@Ralith Ralith merged commit 0fd7327 into master Jan 24, 2022
@Ralith Ralith deleted the docs-omit-self branch January 24, 2022 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants