Skip to content

Fix typo and comma placement in README vLLM comparison - #226

Closed
justink989 wants to merge 1 commit into
NVIDIA:mainfrom
justink989:fix-readme-typo
Closed

justink989 wants to merge 1 commit into
NVIDIA:mainfrom
justink989:fix-readme-typo

Conversation

@justink989

@justink989 justink989 commented Nov 7, 2026 •

Copy link
Copy Markdown

The README's vLLM comparison misspells "engines" as "englines", and the commas in "is not, and will never, be" split the verb phrase in the wrong place.

This PR changes "englines" to "engines" and moves the comma, so the sentence reads: "Hyperflare is not, and will never be, a replacement for production inference engines like vLLM."

The sentence now reads correctly, and its meaning is unchanged.

Validation: this is a documentation-only change to README.md. No tests were run because no code changed.

@original-el8 original-el8 added documentation Improvements or additions to documentation triage:review Proposed change needs code review; this does not imply merge readiness. labels Nov 8, 2026
The README's vLLM comparison misspells "engines" as "englines", and the commas in "is not, and will never, be" split the verb phrase in the wrong place.

Change "englines" to "engines" and move the comma so the sentence reads "is not, and will never be, a replacement for production inference engines like vLLM."

The sentence now reads correctly with its meaning unchanged.

Validation: documentation-only change to README.md; no tests run because no code changed.

Design-Signed-Off-By: Justin Kuhn; design-review=3ad6
Signed-off-by: JustinK <contact@justinkuhn.me>
@original-el8

Copy link
Copy Markdown
Collaborator

corrected by 70b01b5

@justink989
justink989 deleted the fix-readme-typo branch November 13, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation triage:review Proposed change needs code review; this does not imply merge readiness.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants