docs: fix catalog support link in README - #733
Merged
koletzilla merged 2 commits intoAug 28, 2026
Merged
Conversation
koletzilla
self-requested a review
August 28, 2026 15:31
koletzilla
approved these changes
Aug 28, 2026
Contributor
|
Thanks for the PR! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes the broken Catalog Support link in
README.md.The link used a site-root-relative path, which resolves against whichever host renders the file. On GitHub, and on the PyPI project page that renders this README as the package description, it pointed at
https://github.com/integrations/dbt/features-and-configurationsand returned 404. The path was also out of date with respect to the docs site, missing/docsand most of the current path.Replaced it with the absolute URL, matching the documentation link already used earlier in the same README.
Before:
After:
The equivalent paragraph in
docs/index.mdxon thecodex/vendor-dbt-clickhouse-docsbranch (#728) already uses the current path, so this only brings the README copy back in line. A root-relative path stays correct there, since that file is rendered by the docs site, while the README needs the absolute form. The two changes touch different parts ofREADME.mdand do not conflict.Testing
Documentation only, no Python changed, so
make lintand the test suite do not apply. Verified that the previous target returns 404 and that the new URL returns 200 with thecatalog-supportanchor present on the page.Closes #732