Skip to content

London | 26-ITP-May | Zadri Abdule | Sprint 2 | Exercises #1301

Closed
Zadri415 wants to merge 16 commits into
CodeYourFuture:mainfrom
Zadri415:sprint-2/coursework
Closed

London | 26-ITP-May | Zadri Abdule | Sprint 2 | Exercises #1301
Zadri415 wants to merge 16 commits into
CodeYourFuture:mainfrom
Zadri415:sprint-2/coursework

Conversation

@Zadri415

Copy link
Copy Markdown

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

  • address.js — debug fixes
  • author.js — debug fixes
  • recipe.js — debug fixes
  • contains.js — implement contains
  • contains.test.js — tests for contains
  • lookup.js — implement lookup
  • lookup.test.js — tests for lookup
  • querystring.js — robust querystring parser (handles +, %, duplicate keys, values with '=')
  • tally.js — implement tally (frequency counter)
  • tally.test.js — tests for tally
  • invert.js — invert implementation (collects duplicate-value keys into arrays)
  • invert.test.js — tests for invert (including duplicate-values)
  • count-words.js — countWords implementation returning frequency map
  • mode.js — refactored calculateMode into helpers
  • till.js — fixed totalTill implementation (parses coin labels) and exported

@github-actions

Copy link
Copy Markdown

The files changed in this PR don't match what is expected for this task.

Please check that you committed the right files for the task, and that there are no accidentally committed files from other sprints.

Please review the 'files changed' tab at the top of the page.

Here is an example of a file that has been changed on this branch but shouldn't be: Sprint-1/fix/median.js

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

@Zadri415 Zadri415 closed this Jul 26, 2026
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