Skip to content

Fix typos in comments and documentation - #6559

Open
fingolfin wants to merge 1 commit into
masterfrom
claude/spelling-grammar-review-ac603f
Open

Fix typos in comments and documentation#6559
fingolfin wants to merge 1 commit into
masterfrom
claude/spelling-grammar-review-ac603f

Conversation

@fingolfin

Copy link
Copy Markdown
Member

Correct misspellings, a/an errors and doubled words found by a dictionary-based scan of prose in comments, manuals and test files. Files under extern/ are excluded, as are CHANGES.md and doc/manualbib.xml, which record past releases and citations verbatim.

Co-Authored-By: Claude Fable 5.1 noreply@anthropic.com

Correct misspellings, a/an errors and doubled words found by a
dictionary-based scan of prose in comments, manuals and test files.
Files under extern/ are excluded, as are CHANGES.md and
doc/manualbib.xml, which record past releases and citations verbatim.

British spellings, German words in author comments and misspelled
variable names are left untouched. The only identifier-like changes
are two descriptive strings: the "modolo-tail-pcgs-by-list-rep"
method info string and a commented-out %eclaration marker in
doc/ref/pcgs.xml. Some C banner comments are re-justified to keep
their 77-column alignment.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
@fingolfin fingolfin added topic: documentation Issues and PRs related to documentation release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes labels Sep 4, 2026
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 78.98%. Comparing base (acf53f9) to head (91e55ba).

Files with missing lines Patch % Lines
lib/tietze.gi 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6559   +/-   ##
=======================================
  Coverage   78.98%   78.98%           
=======================================
  Files         683      683           
  Lines      294880   294883    +3     
  Branches     8639     8663   +24     
=======================================
+ Hits       232912   232919    +7     
+ Misses      60160    60155    -5     
- Partials     1808     1809    +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Stefan-Kohl

Copy link
Copy Markdown
Member
Screenshot_2026-09-05-07-49-43-142_com android chrome

Why didn't the AI spot the typo in Line 11?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes topic: documentation Issues and PRs related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants