Skip to content

Fix typo: change '1 times' to '1 time' in repeatString test #547

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Shresth2725
Copy link

Because

The test description in repeatString.spec.js had a grammatical typo — it said "1 times" instead of the correct singular "1 time". This can cause confusion and reduces clarity in the tests.

This PR

  • Fixes the typo in the test description on line 10 of repeatString.spec.js.
  • Changes the string from 'repeats the string 1 times' to 'repeats the string 1 time'.

Issue

Closes #546

Additional Information

This is my first open source contribution, happy to learn and improve!

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. 06_repeatString: Fix test description typo
  • 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 this PR includes any changes that affect the solution of an exercise, I've also updated the solution in the /solutions folder (if applicable)

@Shresth2725
Copy link
Author

Hi! Just wanted to check if there’s anything else needed from my side to get this PR merged. Thanks!

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.

Typo in Exercise 06 from TheOdinProject/javascript-exercises
1 participant