Skip to content

HashMap Data Structure : Fixing grammatical error in text#29902

Merged
JoshDevHub merged 2 commits intoTheOdinProject:mainfrom
pocketradio:patch-3
Jul 30, 2025
Merged

HashMap Data Structure : Fixing grammatical error in text#29902
JoshDevHub merged 2 commits intoTheOdinProject:mainfrom
pocketradio:patch-3

Conversation

@pocketradio
Copy link
Contributor

Minor grammatical error in the text

Because

The original had a comma splice, where two independent clauses were incorrectly joined with a comma. The revised version uses a semicolon, which is the grammatically correct choice in this case.

This PR

  • Replaced a comma with a semicolon to fix a comma splice and ensure grammatical correctness.

Issue

Closes #XXXXX

Additional Information

https://www.grammarly.com/blog/punctuation-capitalization/comma-splice/

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added the Content: JavaScript Involves the JavaScript course label Jul 19, 2025
Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

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

Nice find. I do have a small suggestion though given how we tend to stay away from semi colon punctuation.

[ second commit ]

Co-authored-by: Josh Smith <jmsmith1018@gmail.com>
Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

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

Absolutely fine to just press "commit changes"

Thank you for putting this fix in 🚀

@JoshDevHub JoshDevHub merged commit e8ed6ee into TheOdinProject:main Jul 30, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content: JavaScript Involves the JavaScript course

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants