Skip to content

Brussels | ITP-2026-1 | Meysam Razagh | Sprint-2 | Debug-Implement-Interpret#138

Open
Epunch wants to merge 13 commits intoHackYourFutureBelgium:mainfrom
Epunch:Sprint-2
Open

Brussels | ITP-2026-1 | Meysam Razagh | Sprint-2 | Debug-Implement-Interpret#138
Epunch wants to merge 13 commits intoHackYourFutureBelgium:mainfrom
Epunch:Sprint-2

Conversation

@Epunch
Copy link
Copy Markdown

@Epunch Epunch commented May 5, 2026

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

  • contains.js: Implemented logic to check for object properties and passed all unit tests.

  • lookup.js: Created a function to convert an array of country/currency pairs into a lookup object.

  • querystring.js: Fixed a bug regarding multiple equals signs in values and added edge-case testing.

  • tally.js: Implemented a frequency counter for array items with error handling for invalid inputs.

  • invert.js: Fixed a bug involving dynamic key assignment using bracket notation and added missing unit tests.

  • countWords.js (Stretch Goal): Implemented word frequency counting with punctuation removal and case-insensitivity.

Testing

  • All tasks verified using Jest.

  • Ran npx jest to ensure a PASS status for all test suites.

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.

1 participant