Repository navigation
fix: spell product names and acronyms correctly in tag titles - #60
Merged
Merged
Conversation
Posts use lowercase tags, and no tag had a term page, so Hugo built each tag page title from its slug. The UiPath tag page was titled "Uipath", the RPA page "Rpa", and the same happened to API, AutoHotkey, GUI automation, OpenEMR, Power Automate and openadapt-flow. The wrong names showed in page titles, headings, the tag links on each post and the tag RSS feeds. Add a term page under content/tags/<slug>/_index.md for each of these tags that sets the title directly. Add tests/test_tag_titles.py, which runs with the existing unittest suite. It fails if a published post uses a tag that contains an acronym or brand word and has no term page with the expected title, and if a term page has no published posts (that would publish an empty tag page). When Hugo and the theme are present it also builds the site and checks each tag page's <title> and <h1>. Ledger: B238 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The tag title test only understood YAML front matter with LF line endings and raised on anything else. The default archetype writes TOML, so a draft created with `hugo new` would fail CI with "no YAML front matter" before anyone published it. Parse TOML with tomllib, normalize CRLF, accept `draft: True` and trailing comments, and scan every content page instead of only post bundles. Add a few more acronyms that Hugo's title case gets wrong, and report a missing tag page as a test failure instead of a crash. Ledger: B238 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Tag pages on the blog misspelled product names and acronyms. For example, the UiPath tag page was titled "Uipath" and the RPA tag page "Rpa". This pull request gives each affected tag a term page with the correct title, and adds a test that fails when a published post uses an acronym or brand tag that has no term page.
Fixes
content/tags/<slug>/_index.mdfor each of the eight tags, with the titles API, AutoHotkey, GUI Automation, openadapt-flow, OpenEMR, Power Automate, RPA and UiPath.tests/test_tag_titles.pyruns in the existingpython3 -m unittest discover -s testsCI step. It fails if a published post uses an acronym or brand tag that has no term page with the expected title, or if a term page has no published posts, which would publish an empty tag page. It reads both YAML and TOML front matter, so drafts created withhugo newdon't break it. When Hugo and the theme are installed, it also builds the site and checks each tag page's<title>and<h1>.Not fixed here
/tags/still shows every tag by its lowercase slug, for example "uipath" and "rpa". The theme lists terms by slug rather than by title, so the list is consistently lowercase rather than misspelled. Showing titles there would need a theme layout override that changes how every tag looks.How it was tested
main, the new test fails: the source check reports the eight missing term pages, and the build check reports titles such asUipath | OpenAdapt Blog. On this branch, all tests pass.mainand passes here.hugo --minifybuild produces the same set of files asmain. Only the eight tag pages, their RSS feeds,/tags/index.xmland the posts that link to those tags change, and only in tag names.check_benchmark_claims.pyandvalidate_newsletter_render.pypass.fix/blog-overhaul), the full test suite passes. That pull request moves several posts to draft, but each tag that has a term page still has at least one published post.Before merging
rdptag. Before it's published, addcontent/tags/rdp/_index.mdwith the title "RDP" and an"rdp": "RDP"entry inEXPECTED_TITLES, or the test fails..github/workflows/deploy.ymlafterhugo --minifythat runs the build check againstpublic/, so CI also checks the rendered titles. fix: correct post claims, replace the dead signup form, rewrite posts #59 changes that workflow, so this pull request leaves it alone.🤖 Generated with Claude Code