Skip to content

docs: refresh stale numbers in README#35

Merged
Arthur742Ramos merged 1 commit into
mainfrom
arthur742ramos/refresh-readme-numbers
May 15, 2026
Merged

docs: refresh stale numbers in README#35
Arthur742Ramos merged 1 commit into
mainfrom
arthur742ramos/refresh-readme-numbers

Conversation

@Arthur742Ramos

Copy link
Copy Markdown
Owner

Why

The "By the Numbers" table and three other places in the README still quote stats from an old snapshot. With the Copilot SDK Contest Award badge on the README, having stale numbers on the front page is bad PR for the project.

What changes

Stat Before After
Test suite 613 tests 985 tests
Source files 36 TypeScript modules 41 TypeScript modules
Test files 30 Vitest files 60 Vitest files
Lines of code 10,992 TypeScript LOC 11,627 TypeScript LOC

The test-count update touches four places (table, "What Makes It Different" bullet, "Run tests (…)" snippet, and Tech Stack bullet). The other counts only appear in the table.

How I verified

  • Source/test file counts from git ls-files.
  • Test count from npm test on current main.
  • LOC from line count of tracked .ts files under src/.

GitHub stars count is not touched — that's a value the maintainer curates intentionally and shouldn't be drive-by-edited.

Scope

README only. No code or test changes.

The 'By the Numbers' table and three other places quoted the test/source counts from an old snapshot. Bring them in line with the current main branch (post PR #34).

- Test suite: 613 -> 985 tests (updated in 'By the Numbers' table, 'What Makes It Different' bullet, the 'Run tests (...)' comment, and the Tech Stack bullet)

- Source files: 36 -> 41 TypeScript modules

- Test files: 30 -> 60 Vitest files

- Lines of code: 10,992 -> 11,627 TypeScript LOC

Numbers verified against current main via 'git ls-files' and 'npm test' on the local checkout. The GitHub stars count is left as-is since it's a human-curated value the maintainer updates intentionally.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Arthur742Ramos Arthur742Ramos merged commit ecf53d4 into main May 15, 2026
13 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.

1 participant