Skip to content

London | 26-ITP-May | Dipa Sarker | Sprint 2 | Sprint-2-exercises#1272

Open
Dipa-Sarker wants to merge 9 commits into
CodeYourFuture:mainfrom
Dipa-Sarker:Sprint-2-exercises
Open

London | 26-ITP-May | Dipa Sarker | Sprint 2 | Sprint-2-exercises#1272
Dipa-Sarker wants to merge 9 commits into
CodeYourFuture:mainfrom
Dipa-Sarker:Sprint-2-exercises

Conversation

@Dipa-Sarker

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

Completed the Debug, Implement, and Interpret sections of the sprint.

  1. Debug
  • Investigated the bugs in the provided files.
  • Predicted the issues before running the code.
  • Used Node.js to test the predictions and fixed the bugs found.
  1. Implement
    Implemented the required functions and verified them with tests:
  • contains
  • lookup
  • tally
  • querystring

Checked the functions with different inputs to ensure they behave according to the requirements.

  1. Interpret
  • Reviewed the provided code and investigated unfamiliar JavaScript syntax, operators, and functions.
  • Used documentation and console logging to understand how the code works.

**Testing

  • Ran Jest tests to check the implementations.
  • Confirmed that the completed tasks pass the provided test cases.

@Dipa-Sarker Dipa-Sarker added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant