Skip to content

London | 26-ITP-Sep | Mahdi Rafiei | Sprint 3 | implement and rewrite - #1613

Open
mrafeie wants to merge 6 commits into
CodeYourFuture:mainfrom
mrafeie:sprint-3-implement-and-rewrite
Open

mrafeie wants to merge 6 commits into
CodeYourFuture:mainfrom
mrafeie:sprint-3-implement-and-rewrite

Conversation

@mrafeie

@mrafeie mrafeie commented Sep 15, 2026

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

Task code

CYF-1059

Changelist

Sprint 3 | implement and rewrite

Questions

Implemented the getAngleType function to classify angles and added tests for acute, right, obtuse, reflex, and invalid angles.
Implemented the isProperFraction function and added test cases for various scenarios.
Implement getCardValue function to validate and return card values. Added tests for valid and invalid card scenarios.
@mrafeie mrafeie added 📅 Sprint 3 Assigned during Sprint 3 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Structuring-And-Testing-Data The name of the module. labels Sep 15, 2026

@LonMcGregor LonMcGregor left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good work on the implementations you have written, the code is neat and to the point.

However, did you fully test this before opening the PR? I get a few different errors in this directory.

@LonMcGregor LonMcGregor added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Sep 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module-Structuring-And-Testing-Data The name of the module. Reviewed Volunteer to add when completing a review with trainee action still to take. 📅 Sprint 3 Assigned during Sprint 3 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants