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

Update sitemap URL & Disallow #81

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Update sitemap URL & Disallow #81

merged 1 commit into from
Jan 29, 2024

Conversation

msbt
Copy link
Contributor

@msbt msbt commented Jan 29, 2024

Summary of the changes / Why this is an improvement

Updated sitemap URL to non-cratedb.com and also removed the Disallow slash so bots will index the pages (we have/had that in there so the readthedocs URLs won't make it to the search results and compete with cratedb.com URLs.

@msbt msbt requested review from amotl and matkuliak January 29, 2024 10:40
@amotl amotl merged commit 47b0e7c into main Jan 29, 2024
4 checks passed
@amotl amotl deleted the msbt-sitemap-update branch January 29, 2024 12:25
@amotl
Copy link
Member

amotl commented Jan 29, 2024

also removed the Disallow slash

Thank you. Disallow: / is still ok, at, for example, https://github.com/crate/crate/pull/15464/files?

@msbt
Copy link
Contributor Author

msbt commented Jan 29, 2024

@amotl yes, because we specifically don't want all other repos indexed by bots, this the only one that is supposed to (to avoid having the readthedocs pages to be in the search results, we've had that before that they were sometimes ranked higher than the actual website)

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