Skip to content

WESTMIDLAND ITP-MAY-2025 AHMAD EHSAS Coursework/sprint-2 #630

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

WESTMIDLAND ITP-MAY-2025 AHMAD EHSAS Coursework/sprint-2 #630

wants to merge 9 commits into from

Conversation

ahmadehsas
Copy link

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with REGION | COHORT_NAME | FIRST_NAME LAST_NAME | PROJ_NAME
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

Completed all the mandatory coursework, including all exercises and errors, as well as interpret and stretch-explore.

@ahmadehsas
Answered comment for count update in exercise -1
65fe894
@ahmadehsas
predicted and explained the error. and finally, fixed the code.
c2e158f
@ahmadehsas
the error is predicted before running the code and then explained th…
5e632d2 
…e error. and finally, rewrote the code to fix the problem.
@ahmadehsas
explained the technical error and fixed the code as comment.
4cdfdd4
@ahmadehsas
The sum function is undefined because the function return was writte…
eb5e0dc 
…n incorrectly. i explained the error and rewrote the code as a comment.
@ahmadehsas
in this code the function is not working properly. I explained the i…
a696e17 
…ssue and fixed the code.
@ahmadehsas
i returned the Body Mass Index(BMI) of myselfin return function and t…
84ed53e 
…hen explained it in comment how to calculate the BMI.
@ahmadehsas
i turned the code into reusable block of code and i called the functi…
7a09127 
…on many times for different inputs.
@ahmadehsas
I answered all the questions and python visualiser helped me answer t…
f813ae6 
…hese questions.
@ahmadehsas ahmadehsas added the Needs Review Participant to add when requesting review label Jul 1, 2025
@Bluejay600 Bluejay600 added Reviewed Volunteer to add when completing a review and removed Reviewed Volunteer to add when completing a review labels Jul 1, 2025
@kodnificent kodnificent added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. 📅 Sprint 2 Assigned during Sprint 2 of this module and removed Needs Review Participant to add when requesting review labels Jul 2, 2025
@kodnificent kodnificent self-requested a review July 2, 2025 07:47
kodnificent
kodnificent suggested changes Jul 2, 2025
View reviewed changes
Copy link

@kodnificent kodnificent left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ahmadehsas This looks good. It is only missing 3-mandatory-implement/2-cases.js.

Can you check it out?

Sprint-2/1-key-errors/0.js
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also write your prediction here?

@kodnificent kodnificent added Reviewed Volunteer to add when completing a review and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kodnificent kodnificent kodnificent requested changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Reviewed Volunteer to add when completing a review 📅 Sprint 2 Assigned during Sprint 2 of this module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ahmadehsas @kodnificent @Bluejay600