Skip to content

Emit canonical URLs for all rustdoc pages #1792

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

Merged
merged 2 commits into from
Aug 26, 2022

Conversation

jsha
Copy link
Contributor

@jsha jsha commented Aug 9, 2022

Previously, we did not emit canonical URLs for crates that had an alternate documentation URL. However, this produces bad results for those crates, with older versions being selected as canonical even if the latest version uses docs.rs as its documentation URL.

Part of #74

@github-actions github-actions bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Aug 9, 2022
Copy link
Member

@syphar syphar left a comment

Choose a reason for hiding this comment

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

code looks good, with the matching explanation in the issues.

I can't test locally today, if no one else can do this until monday/tuesday I'll test/merge/deploy then

Previously, we did not emit canonical URLs for crates that had an
alternate documentation URL. However, this produces bad results for
those crates, with older versions being selected as canonical even
if the latest version uses docs.rs as its documentation URL.
@jsha jsha force-pushed the canonical-for-all branch from 9c81a2f to ebaf774 Compare August 11, 2022 19:27
@jsha
Copy link
Contributor Author

jsha commented Aug 18, 2022

Ping on this? :-)

@syphar
Copy link
Member

syphar commented Aug 26, 2022

LGTM, sorry for the delay

@syphar syphar merged commit d226151 into rust-lang:master Aug 26, 2022
@github-actions github-actions bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Aug 26, 2022
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Aug 27, 2022
@jsha jsha mentioned this pull request Aug 31, 2022
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.

3 participants