Skip to content

London | 26-ITP-Jan | Ebrahim Moqbel | Sprint 2 | coursework/sprint-2#1099

Open
Ebrahim-Moqbel wants to merge 9 commits intoCodeYourFuture:mainfrom
Ebrahim-Moqbel:acoursework/sprint-2
Open

London | 26-ITP-Jan | Ebrahim Moqbel | Sprint 2 | coursework/sprint-2#1099
Ebrahim-Moqbel wants to merge 9 commits intoCodeYourFuture:mainfrom
Ebrahim-Moqbel:acoursework/sprint-2

Conversation

@Ebrahim-Moqbel
Copy link

Learners, PR Template

Self checklist

  • [ x] I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • [ x] My changes meet the requirements of the task
  • [ x] I have tested my changes
  • [ x] My changes follow the style guide

Changelist

I completed the following filles :

  • key errors
  • mandatory debug
  • mandatory implement
  • mandatory interpret

@github-actions

This comment has been minimized.

@Ebrahim-Moqbel Ebrahim-Moqbel changed the title London | 26-ITP-Jan | Ebrahim Moqbel | Sprint 1 | coursework/sprint-2 London | 26-ITP-Jan | Ebrahim Moqbel | Sprint 2 | coursework/sprint-2 Feb 28, 2026
@Ebrahim-Moqbel Ebrahim-Moqbel added 🏕 Priority Mandatory This work is expected 📅 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 Feb 28, 2026
@cjyuan cjyuan 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 Mar 7, 2026
Copy link
Contributor

@cjyuan cjyuan left a comment

Choose a reason for hiding this comment

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

Code is not consistently formatted.

Consider install prettier VSCode extension and enable formatting on save/paste on VSCode
as recommended in
https://github.com/CodeYourFuture/Module-Structuring-and-Testing-Data/blob/main/readme.md

Then, resave all your files (in Sprint 2) to ensure all code is consistently formatted.

@cjyuan cjyuan 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 Mar 7, 2026
@Ebrahim-Moqbel Ebrahim-Moqbel added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 7, 2026
@Ebrahim-Moqbel
Copy link
Author

I have installed prettier to VS code and selected the copy-paste save format. I have been reading about reverting and reseting commits in git. I know how to. revert a specific commit but I still need to know how can I select all the commits.

I was wondering if you need me revert all of them and commit them again. Also After installing prettier I still find the format the same and I think it's consistent.

@cjyuan cjyuan removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 7, 2026
@cjyuan
Copy link
Contributor

cjyuan commented Mar 9, 2026

If you have enabled "Format on save" but it is not working, it is likely that you haven't assign a formatter for JS file. This could happen if you have zero or multiple extensions that can format .js file.

If you have installed "Prettier" extension. To assign it as the formatter of JS code, you can try:
(1) Use "Format document" to format the JS file. Sometimes, VSCode will ask you to choose a formatter, and you can manually select "Prettier".
(2) Edit settings.json and set Prettier as the default formatter for JS.
See: https://marketplace.visualstudio.com/items?itemName=esbenp.prettier-vscode


Can you also address all the follow-up questions in the inline comments.

@Ebrahim-Moqbel
Copy link
Author

I have addressed the questions above and regarding prettier I have installed it and selected the the format to be on paste as well but I may still need to Edit settings.json but I am still not sure how to access to this settings whether they are in an external file or in built settings in VS code.

In the bellow video prettier asked for configuration and I am hoping you could advise me what I expected to do at this stage. Thanks xx

Screenshot 2026-02-26 at 20 57 19 (2)

@Ebrahim-Moqbel Ebrahim-Moqbel added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 10, 2026
@cjyuan
Copy link
Contributor

cjyuan commented Mar 10, 2026

I don't see any video. I only see an image.

Prettier won't format the code if there is a syntax error in the code.

The file settings.json is located at
$HOME/Library/Application\ Support/Code/User/settings.json.

If you still have problem setting Prettier as your default formatter, try to seek for assistance on Saturday in the workshop?


Thanks for the responses to each comment. All good now.

@cjyuan cjyuan removed 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. labels Mar 10, 2026
@cjyuan cjyuan added the Complete Volunteer to add when work is complete and all review comments have been addressed. label Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed. Module-Structuring-And-Testing-Data The name of the module. 🏕 Priority Mandatory This work is expected 📅 Sprint 2 Assigned during Sprint 2 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants