Skip to content

fix: correct typos in pekko-http docs and tests#1075

Merged
He-Pin merged 2 commits into
apache:mainfrom
He-Pin:fix/typo-fixes
Jun 19, 2026
Merged

fix: correct typos in pekko-http docs and tests#1075
He-Pin merged 2 commits into
apache:mainfrom
He-Pin:fix/typo-fixes

Conversation

@He-Pin

@He-Pin He-Pin commented Jun 18, 2026

Copy link
Copy Markdown
Member

Motivation

Typos found in documentation markdown files and test code.

Modification

Fixed typos in docs: AppacheApache, bellowbelow, programaticallyprogrammatically, nooneno one. Fixed typos in tests: decodeingdecoding, nextingnesting.

Result

Cleaner codebase with corrected spelling.

Tests

Not run - typo fixes only (docs and test strings)

References

None - typo cleanup

Motivation:
Typos found in documentation markdown files and test code.

Modification:
- Appache→Apache in timeouts.md (2 occurrences)
- bellow→below in host.md
- programatically→programmatically in extractMethod.md
- noone→no one in security directives docs
- decodeing→decoding in CodingDirectivesSpec.scala
- nexting→nesting in PathDirectivesTest.java

Result:
Cleaner codebase with corrected spelling.

Tests:
Not run - typo fixes only (docs and test strings)

References:
None - typo cleanup
Comment thread docs/src/main/paradox/routing-dsl/directives/security-directives/index.md Outdated
Motivation:
CR review comment from pjfanning: "noone is correct as is".

Modification:
Revert the noone → no one change in security-directives/index.md.

Result:
Original spelling preserved per maintainer feedback.

Tests:
Not run - doc string revert only

References:
Fixes apache#1075 (comment)
@He-Pin He-Pin requested a review from pjfanning June 19, 2026 05:06

@pjfanning pjfanning left a comment

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.

lgtm

@He-Pin He-Pin merged commit aec584c into apache:main Jun 19, 2026
5 checks passed
@He-Pin He-Pin deleted the fix/typo-fixes branch June 19, 2026 08:42
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.

2 participants