Fix the typos in the opening sentence of insight #3 - #329
Merged
Merged
Conversation
"you always havte to fullfll certain requoirememnts" - three typos in the first sentence of the article, which is also the sentence the site's generated description and llms.txt take from the page. Nothing catches this: the spelling gate names British forms in a list, so a word that is neither spelling reads as prose it has no opinion about. "these can be" and "most of them" now agree with the plural the sentence gained, and the line is wrapped back to the column the file is set in. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0182AiPodwfGRKPNZG9D5epe
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
2 participants
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.
Follow-up to the direct edit in
2a24ce5, which left three typos in the article's first sentence:Now:
Two agreement fixes came with it, because the subject went from singular to plural in that edit:
that can be→these can be, andmost of it→most of them. The line was also back as a single 106-character line and is wrapped to the column the file is set in.Why this one is worth noticing
The first sentence of a page is what
scripts/lib/pages.mjsbuilds the page description from, so it is also what stands in the search index, inllms.txtand under the card — the typo would have shown up in four places, not one.No gate catches it.
test/spelling.test.mjsnames British forms from a list; a word that is neither spelling is prose it has no opinion about. Verified: all 251 tests pass on the state with the typos in it, and the deploy from it was green.Gates
test(251),check:conventions,check:examples,check:api-namesanddocs:buildpass locally.Carried over unchanged from
2a24ce5*The entity is just an example*and*The table is just an example*— all three captions are now built the same way; the BAPI one already read like that.🤖 Generated with Claude Code
https://claude.ai/code/session_0182AiPodwfGRKPNZG9D5epe
Generated by Claude Code