Skip to content

Glasgow | 26-ITP-Jan | Joma Alrzini | Sprint 2 | Coursework#1060

Open
Alrzini2025 wants to merge 7 commits intoCodeYourFuture:mainfrom
Alrzini2025:Sprint-2
Open

Glasgow | 26-ITP-Jan | Joma Alrzini | Sprint 2 | Coursework#1060
Alrzini2025 wants to merge 7 commits intoCodeYourFuture:mainfrom
Alrzini2025:Sprint-2

Conversation

@Alrzini2025
Copy link

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

changes in my sprint 2

Questions

no questions.

@github-actions

This comment has been minimized.

@Alrzini2025 Alrzini2025 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 25, 2026
@Edu-Vin Edu-Vin added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Feb 25, 2026
@@ -1,13 +1,13 @@
// Predict and explain first...
// =============> write your prediction here
// Missing backticks for template string template strings need to use backticks `, but here ${} is used without them
Copy link

Choose a reason for hiding this comment

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

There were backticks in the original code.

Copy link

Choose a reason for hiding this comment

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

Kindly revisit this file to make it clearer. Leave the comments on the original code.

Copy link

Choose a reason for hiding this comment

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

You didn't write tests for your fix.

@Edu-Vin Edu-Vin added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Reviewed Volunteer to add when completing a review with trainee action still to take.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants