Skip to content

Manchester | 26-ITP-Jan | khalidbih | Sprint 3 | coursework/sprint-3-implement-and-rewrite#1268

Closed
khalidbih wants to merge 6 commits intoCodeYourFuture:mainfrom
khalidbih:sprint-3-implement-and-rewrite-updated
Closed

Manchester | 26-ITP-Jan | khalidbih | Sprint 3 | coursework/sprint-3-implement-and-rewrite#1268
khalidbih wants to merge 6 commits intoCodeYourFuture:mainfrom
khalidbih:sprint-3-implement-and-rewrite-updated

Conversation

@khalidbih
Copy link

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 :

I have completed the coursework/sprint-3-implement-and-rewrite.

@khalidbih khalidbih 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 Mar 15, 2026
@khalidbih
Copy link
Author

@cjyuan Hi, I’ve created a new branch from the latest upstream main and updated all exercises in Sprint-3/1-implement-and-rewrite-tests/ according to the latest spec. I hope I’ve done it correctly and this PR.

@khalidbih
Copy link
Author

@cjyuan Hi, I’ve created a new branch from the latest upstream main and updated all exercises in Sprint-3/1-implement-and-rewrite-tests/ according to the latest spec. I hope I’ve done it correctly with this PR. It looks like there are still conflicts

@cjyuan
Copy link
Contributor

cjyuan commented Mar 15, 2026

This branch is still 10 commits behind CYF's main, meaning it is not up-to-date. Can you use "Sync Fork" on your main branch to ensure your main is updated, then recreate this branch from your main?

@cjyuan cjyuan 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 Mar 15, 2026
@khalidbih
Copy link
Author

This branch is still 10 commits behind CYF's main, meaning it is not up-to-date. Can you use "Sync Fork" on your main branch to ensure your main is updated, then recreate this branch from your main?

Ok, I will do it !

@khalidbih khalidbih closed this Mar 16, 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