Skip to content

London | 26-ITP-May | Rizqah Popoola | Sprint 2 | Coursework#1433

Open
risikatpopoola wants to merge 5 commits into
CodeYourFuture:mainfrom
risikatpopoola:coursework/Sprint2
Open

London | 26-ITP-May | Rizqah Popoola | Sprint 2 | Coursework#1433
risikatpopoola wants to merge 5 commits into
CodeYourFuture:mainfrom
risikatpopoola:coursework/Sprint2

Conversation

@risikatpopoola

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

Changelist

This PR is for coursework sprint 2

@risikatpopoola risikatpopoola added 📅 Sprint 2 Assigned during Sprint 2 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 Jul 1, 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 start on these tasks. Could you take another look at the first couple?

Also, remember to attempt the 4th interpreting task

// =============> write your new code here
// Te error message says 'str' has already been declared so we can use another name for our
//output to solve that error we change the output variable name
// function capitalise(str) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can you try implementing this change?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have done this, thank you

Comment thread Sprint-2/1-key-errors/1.js Outdated
console.log(decimalNumber);
// Finally, correct the code to fix the problem
// function convertToPercentage(decimalNumber) {
// const decimalNumber1 = 0.5;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If you try to run this code, does it work as expected?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have rewritten the code and it now works as expected

@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 Jul 6, 2026
@risikatpopoola risikatpopoola added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Jul 12, 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. Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Reviewed Volunteer to add when completing a review with trainee action still to take. 📅 Sprint 2 Assigned during Sprint 2 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants