Skip to content

[DOCS-15413] Add prodname shortcode BlockIgnore to Vale config - #39194

Closed
hestonhoffman wants to merge 2 commits into
masterfrom
heston/prodname
Closed

[DOCS-15413] Add prodname shortcode BlockIgnore to Vale config#39194
hestonhoffman wants to merge 2 commits into
masterfrom
heston/prodname

Conversation

@hestonhoffman

Copy link
Copy Markdown
Collaborator

What does this PR do? What is the motivation?

Fixes DOCS-15413

Adds a BlockIgnore to .vale.ini so text already wrapped in the prodname shortcode is skipped by the new Datadog.Prodname Vale rule (see datadog-vale#98), instead of being flagged again.

This PR is a draft — remaining work on this branch includes running the new Vale rule against existing content and wrapping flagged product names in the prodname shortcode.

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Used Claude Code to draft the Vale rule design, the .vale.ini BlockIgnore, and this PR.

Additional notes

@jhgilbert jhgilbert added the astro-reorg-no-conflicts Needs manual conflict resolution after replatforming reorg label Aug 17, 2026
@hestonhoffman

Copy link
Copy Markdown
Collaborator Author

Closing in favor of a fresh PR — this branch predates the Hugo directory reorg and picked up unrelated diff noise. Re-opening the code-only prodname changes on a new branch off current master.

@hestonhoffman
hestonhoffman deleted the heston/prodname branch August 18, 2026 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astro-reorg-no-conflicts Needs manual conflict resolution after replatforming reorg

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants