Skip to content

Docs: remove hardcoded table of event types in favour of autogenerated one#99566

Merged
alexey-milovidov merged 1 commit intomasterfrom
Blargian-patch-22
Mar 16, 2026
Merged

Docs: remove hardcoded table of event types in favour of autogenerated one#99566
alexey-milovidov merged 1 commit intomasterfrom
Blargian-patch-22

Conversation

@Blargian
Copy link
Member

Remove hardcoded table in favour of autogenerated one introduced in ClickHouse/clickhouse-docs#5749

Changelog category (leave one):

  • Documentation (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

Remove hardcoded table in favour of autogenerated one introduced in ClickHouse/clickhouse-docs#5749
@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Mar 16, 2026

Workflow [PR], commit [1125aaf]

Summary:


AI Review

Summary

This PR removes the hardcoded system.events event-type table from docs/en/operations/system-tables/events.md and leaves the autogenerated section markers as the source of truth. I did not find correctness, safety, or rollout issues in the submitted change and have no blocker/major findings.

Missing context

  • ⚠️ No CI/doc-build artifacts were provided in the review request, so I could not verify rendered docs output for this PR directly.

ClickHouse Rules

Item Status Notes
Deletion logging
Serialization versioning
Core-area scrutiny
No test removal
Experimental gate
No magic constants
Backward compatibility
SettingsHistory.cpp
Safe rollout
Compilation time

Final Verdict

  • Status: ✅ Approve

@clickhouse-gh clickhouse-gh bot added the pr-documentation Documentation PRs for the specific code PR label Mar 16, 2026
@alexey-milovidov alexey-milovidov self-assigned this Mar 16, 2026
@alexey-milovidov alexey-milovidov added this pull request to the merge queue Mar 16, 2026
Merged via the queue into master with commit d0603c3 Mar 16, 2026
163 checks passed
@alexey-milovidov alexey-milovidov deleted the Blargian-patch-22 branch March 16, 2026 10:21
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-documentation Documentation PRs for the specific code PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants