Skip to content

Birmingham | ITP Jan 2026 | Arunkumar Akilan | Sprint 2 | Data Groups#1025

Closed
arunkumarakilan wants to merge 14 commits intoCodeYourFuture:mainfrom
arunkumarakilan:coursework/sprint2
Closed

Birmingham | ITP Jan 2026 | Arunkumar Akilan | Sprint 2 | Data Groups#1025
arunkumarakilan wants to merge 14 commits intoCodeYourFuture:mainfrom
arunkumarakilan:coursework/sprint2

Conversation

@arunkumarakilan
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

🐞 Debug

  • Fixed bugs in address.js
  • Fixed bugs in author.js
  • Fixed bugs in recipe.js
  • Ensured all debug tests pass

⚙️ Implementation

  • Implemented tally function to count occurrences in an array
  • Implemented contains function to check if object has a property
  • Implemented createLookup function to convert key-value pairs into an object
  • Implemented parseQueryString with edge case handling
  • Implemented invert function to swap keys and values

🧪 Testing

  • Added unit tests for all functions
  • Covered edge cases (empty input, invalid input, multiple values, etc.)
  • All tests are passing

Notes

  • Used methods like Object.keys, Object.entries, and array iteration
  • Improved understanding of objects and key-value handling

Questions

No questions at the moment

@github-actions
Copy link
Copy Markdown

The changed files 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 changed files tab at the top of the page, we are only expecting changes in this directory: ^Sprint-2/

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.

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