Skip to content

Conversation

@dhtclk
Copy link
Collaborator

@dhtclk dhtclk commented Nov 20, 2025

Summary

Auditing Vale Rules and Including a Report Script

For now disabling sentence length and future tense rules. They're very subjective and flag too many false positives, so for now I've opted to exclude them in order to tackle other issues until we decide to either reinstate, remove, or refine them.

Checklist

@vercel
Copy link

vercel bot commented Nov 20, 2025

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

Project Deployment Preview Comments Updated (UTC)
clickhouse-docs Ready Ready Preview Comment Nov 20, 2025 8:03pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
clickhouse-docs-jp Ignored Ignored Nov 20, 2025 8:03pm
clickhouse-docs-ru Ignored Ignored Preview Nov 20, 2025 8:03pm
clickhouse-docs-zh Ignored Ignored Preview Nov 20, 2025 8:03pm

@dhtclk dhtclk marked this pull request as ready for review November 20, 2025 16:21
@dhtclk dhtclk requested a review from a team as a code owner November 20, 2025 16:21
Copy link
Member

@Blargian Blargian left a comment

Choose a reason for hiding this comment

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

Approved but should run on .md and .mdx

@dhtclk dhtclk merged commit 80c32f7 into main Nov 21, 2025
15 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.

3 participants