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

Revert "Only add a canonical tag to the latest version of man pages" #1455

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

deivid-rodriguez
Copy link
Member

This reverts commit 50063c7.

What was the end-user problem that led to this PR?

The problem was that when you google bundle check, the first result is https://bundler.io/v2.0/man/bundle-check.1.html. Instead, the first result would be ideally the man page for the latest version, not for a version no longer supported.

What was your diagnosis of the problem?

My diagnosis was that we should tell google to consider the non versioned URL the canonical page for all versioned ones, so that it always prefer to list that page.

What is your fix for the problem, implemented in this PR?

My fix is to revert the commit where I stopped doing this. The rationale was that these pages are not identical to each other, but I think canonical is also appropriate for pages with similar content.

Why did you choose this fix out of the possible options?

I chose this fix because I'd like to see if has the desired effect.

@deivid-rodriguez deivid-rodriguez force-pushed the deivid-rodriguez/canonical branch from e98289f to 8fc9ea8 Compare February 13, 2025 10:37
@deivid-rodriguez deivid-rodriguez merged commit ae88f03 into main Feb 13, 2025
3 checks passed
@deivid-rodriguez deivid-rodriguez deleted the deivid-rodriguez/canonical branch February 13, 2025 10: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.

1 participant