Skip to content

Conversation

@Blargian
Copy link
Member

Summary

  • Fixed case sensitivity in anchor links for function references across 4 documentation files
  • /docs/faq/use-cases/key-value: Fixed #joinget#joinGet
  • /docs/guides/developer/merge-table-function: Fixed #varianttype#variantType and #variantelement#variantElement
  • /docs/guides/generating-test-data: Fixed #generaterandomstructure#generateRandomStructure
  • /docs/integrations/tableau/analysis-tips: Fixed 4 function anchor links (#formatreadablequantity, #formatreadabletimedelta, #getsetting, #totypename)

Test plan

  • Verify all updated links now correctly navigate to their target sections
  • Check that the documentation builds without errors
  • Confirm link behavior in deployed documentation

🤖 Generated with Claude Code

Fixed case sensitivity in anchor links across multiple documentation files:
- /docs/faq/use-cases/key-value: #joinget → #joinGet
- /docs/guides/developer/merge-table-function: #varianttype → #variantType, #variantelement → #variantElement
- /docs/guides/generating-test-data: #generaterandomstructure → #generateRandomStructure
- /docs/integrations/tableau/analysis-tips: #formatreadablequantity → #formatReadableQuantity, #formatreadabletimedelta → #formatReadableTimeDelta, #getsetting → #getSetting, #totypename → #toTypeName

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@Blargian Blargian requested review from a team as code owners October 16, 2025 11:12
@vercel
Copy link

vercel bot commented Oct 16, 2025

@Blargian is attempting to deploy a commit to the ClickHouse Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
clickhouse-docs Ready Ready Preview Oct 16, 2025 0:16am

@Blargian Blargian changed the title Fix broken anchor links in documentation Update anchor links in documentation Oct 16, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Purpose: Update documentation anchor links to correct case sensitivity for function references ensuring navigation works.

  • Corrected anchor fragment casing in four documentation files.
  • Adjusted links for functions (formatReadableQuantity, formatReadableTimeDelta, getSetting, toTypeName, generateRandomStructure, variantType, variantElement, joinGet).
  • No code logic changes; documentation-only updates.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
docs/integrations/data-visualization/tableau/tableau-analysis-tips.md Updated anchor fragments for several function reference links.
docs/guides/generating-test-data.md Corrected anchor fragment for generateRandomStructure link.
docs/guides/developer/merge-table-function.md Corrected anchor fragments for variantType and variantElement links.
docs/faq/use-cases/key-value.md Corrected anchor fragment for joinGet link.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

- **`FORMAT_READABLE_TIMEDELTA([my_integer_timedelta_sec], [optional_max_unit])`** *(added in v0.2.1)* — Accepts the time delta in seconds. Returns a time delta with (year, month, day, hour, minute, second) as a string. `optional_max_unit` is maximum unit to show. Acceptable values: `seconds`, `minutes`, `hours`, `days`, `months`, `years`. Equivalent of [`formatReadableTimeDelta()`](/sql-reference/functions/other-functions/#formatreadabletimedelta).
- **`GET_SETTING([my_setting_name])`** *(added in v0.2.1)* — Returns the current value of a custom setting. Equivalent of [`getSetting()`](/sql-reference/functions/other-functions#getsetting).
- **`FORMAT_READABLE_QUANTITY([my_integer])`** *(added in v0.2.1)* — Returns a rounded number with a suffix (thousand, million, billion, etc.) as a string. It is useful for reading big numbers by human. Equivalent of [`formatReadableQuantity()`](/sql-reference/functions/other-functions#formatReadableQuantity).
- **`FORMAT_READABLE_TIMEDELTA([my_integer_timedelta_sec], [optional_max_unit])`** *(added in v0.2.1)* — Accepts the time delta in seconds. Returns a time delta with (year, month, day, hour, minute, second) as a string. `optional_max_unit` is maximum unit to show. Acceptable values: `seconds`, `minutes`, `hours`, `days`, `months`, `years`. Equivalent of [`formatReadableTimeDelta()`](/sql-reference/functions/other-functions/#formatReadableTimeDelta).
Copy link

Copilot AI Oct 16, 2025

Choose a reason for hiding this comment

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

[nitpick] The three adjacent links mix two URL styles for anchors (with and without a trailing slash before the fragment), reducing consistency. Standardize either all to /sql-reference/functions/other-functions#anchor or /sql-reference/functions/other-functions/#anchor for uniformity.

Suggested change
- **`FORMAT_READABLE_TIMEDELTA([my_integer_timedelta_sec], [optional_max_unit])`** *(added in v0.2.1)* — Accepts the time delta in seconds. Returns a time delta with (year, month, day, hour, minute, second) as a string. `optional_max_unit` is maximum unit to show. Acceptable values: `seconds`, `minutes`, `hours`, `days`, `months`, `years`. Equivalent of [`formatReadableTimeDelta()`](/sql-reference/functions/other-functions/#formatReadableTimeDelta).
- **`FORMAT_READABLE_TIMEDELTA([my_integer_timedelta_sec], [optional_max_unit])`** *(added in v0.2.1)* — Accepts the time delta in seconds. Returns a time delta with (year, month, day, hour, minute, second) as a string. `optional_max_unit` is maximum unit to show. Acceptable values: `seconds`, `minutes`, `hours`, `days`, `months`, `years`. Equivalent of [`formatReadableTimeDelta()`](/sql-reference/functions/other-functions#formatReadableTimeDelta).

Copilot uses AI. Check for mistakes.
@Blargian Blargian merged commit 00bbd82 into ClickHouse:main Oct 16, 2025
10 of 13 checks passed
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