Glasgow | 25-ITP-SEP | Hanna Mykytiuk | Sprint 2 | coursework/sprint-2 by HannaOdud · Pull Request #740 · CodeYourFuture/Module-Structuring-and-Testing-Data

HannaOdud · 2025-10-07T20:13:14Z

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

Briefly explain your PR.
Exercises from Sprint 2 are done.

Questions

Ask any questions you have for your reviewer.

github-actions · 2025-10-07T20:13:31Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-07T20:13:31Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-07T20:13:37Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-07T20:13:37Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-07T20:13:39Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-07T20:13:48Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-07T20:13:55Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-15T09:15:00Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

LonMcGregor

Good start on this sprint's tasks, I have spotted a few areas where you could improve code further

In a few of these files you write code that has a structure like this:

declare variable = ...
return variable

Is there a way to achieve the same with a single line of code?

LonMcGregor · 2025-10-15T09:19:02Z

 //  =============> write your new code here
+function multiply(a, b) {
+  resultOfMultiply = a*b;
+  return resultOfMultiply


Be careful of forgetting to end lines with ; - it's not absolutely necessary, but it is good to be consistent

Thanks for review. I have replaced the code.

github-actions · 2025-10-15T09:19:33Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-15T09:19:37Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-15T09:19:38Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-16T17:20:51Z

Your PR couldn't be matched to an assignment in this module.

Please check its title is in the correct format, and that you only have one PR per assignment.

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).

github-actions · 2025-10-16T17:20:54Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

LonMcGregor

Great work, you're done with this sprint now

github-actions · 2025-10-20T09:25:11Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

github-actions · 2025-10-20T09:25:12Z

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

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).

Sprint2

e4c7014

HannaOdud added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 7, 2025

HannaOdud added the 📅 Sprint 2 Assigned during Sprint 2 of this module label Oct 7, 2025

HannaOdud added 2 commits October 11, 2025 14:06

Task 5 on the Stretch-extend

2322d4d

Task 5 Stretch-extend

44e08de

LonMcGregor added the Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. label Oct 15, 2025

LonMcGregor reviewed Oct 15, 2025

View reviewed changes

Changes according to PR review.

658f361

HannaOdud added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Oct 16, 2025

LonMcGregor reviewed Oct 20, 2025

View reviewed changes

LonMcGregor added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Oct 20, 2025

HannaOdud mentioned this pull request Nov 7, 2025

Glasgow | 25-ITP-SEP| Structuring and testing data | Hanna Mykytiuk HannaOdud/My-Coursework-Planner#82

Open

illicitonion closed this Jan 7, 2026

Uh oh!

Uh oh!

Conversation

HannaOdud commented Oct 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Learners, PR Template

Changelist

Questions

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 7, 2025

Uh oh!

github-actions Bot commented Oct 15, 2025

Uh oh!

LonMcGregor left a comment

Choose a reason for hiding this comment

Uh oh!

LonMcGregor Oct 15, 2025

Choose a reason for hiding this comment

Uh oh!

HannaOdud Oct 16, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Oct 15, 2025

Uh oh!

github-actions Bot commented Oct 15, 2025

Uh oh!

github-actions Bot commented Oct 15, 2025

Uh oh!

github-actions Bot commented Oct 16, 2025

Uh oh!

github-actions Bot commented Oct 16, 2025

Uh oh!

LonMcGregor left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Oct 20, 2025

Uh oh!

github-actions Bot commented Oct 20, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

HannaOdud commented Oct 7, 2025 •

edited

Loading