Skip to content

docs: fix typos across documentation - #378

Merged
UlisesGascon merged 1 commit into
expressjs:masterfrom
vaibhavmashal:docs/fix-readme-typos-and-examples
Aug 30, 2026
Merged

docs: fix typos across documentation#378
UlisesGascon merged 1 commit into
expressjs:masterfrom
vaibhavmashal:docs/fix-readme-typos-and-examples

Conversation

@vaibhavmashal

Copy link
Copy Markdown
Contributor

Description

This PR improves documentation clarity and fixes minor typos across the repository:

  1. README.md:

    • Updates deprecated node-uuid package reference to standard uuid in the custom token example.
    • Fixes grammar and typos in documentation:
      • "a requests" -> "requests" (immediate option description)
      • "of it's choosing" -> "of its choosing" (token description)
      • "if exists" -> "if it exists" (:referrer and :url tokens)
      • "Tokens are references by" -> "Tokens are referenced by"
      • "tokens is object" -> "tokens is an object"
      • "log all request" -> "log all requests" (examples header descriptions)
  2. index.js:

    • Fixes typo in total-time token comment ("cover to milliseconds" -> "convert to milliseconds").
  3. test/morgan.js:

    • Fixes test title typo ("should color 4xx yelow" -> "should color 4xx yellow").

Test plan

  • Ran npm test (all 99 tests passing across mocha test suites).
  • Ran npm run lint (eslint passing with no errors or warnings).

Copilot AI lite review requested due to automatic review settings August 30, 2026 05:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread README.md
```js
var express = require('express')
var morgan = require('morgan')
var uuid = require('node-uuid')

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Catch!

@UlisesGascon UlisesGascon self-assigned this Aug 30, 2026
@UlisesGascon UlisesGascon changed the title docs: fix typos in README, comments, and test descriptions, and update uuid example docs: fix typos across documentation Aug 30, 2026
@UlisesGascon
UlisesGascon merged commit 5a5902a into expressjs:master Aug 30, 2026
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants