Skip to content

London |SDC- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls#335

Open
jamallaqdiem wants to merge 18 commits intoCodeYourFuture:mainfrom
jamallaqdiem:individual-shell-tools-exercises
Open

London |SDC- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls#335
jamallaqdiem wants to merge 18 commits intoCodeYourFuture:mainfrom
jamallaqdiem:individual-shell-tools-exercises

Conversation

@jamallaqdiem
Copy link

@jamallaqdiem jamallaqdiem commented Mar 8, 2026

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

Completed all exercises about command lines.

@github-actions

This comment has been minimized.

@jamallaqdiem jamallaqdiem changed the title London| 2026-MAR-SDC | Jamal Laqdiem| Sprint 1| Individual Shell Tools London | CDS- MARC-2026 | JAMAL LAQDIEM | sprint 1 | Individual Shell Tolls Mar 8, 2026
@github-actions

This comment has been minimized.

9 similar comments
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@jamallaqdiem jamallaqdiem changed the title London | CDS- MARC-2026 | JAMAL LAQDIEM | sprint 1 | Individual Shell Tolls London | CDS- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls Mar 8, 2026
@github-actions

This comment has been minimized.

@jamallaqdiem jamallaqdiem changed the title London | CDS- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls London |SDS- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls Mar 9, 2026
@jamallaqdiem jamallaqdiem changed the title London |SDS- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls London |SDC- MARC-2026 | JAMAL LAQDIEM | Sprint 1 | Individual Shell Tolls Mar 9, 2026
@jamallaqdiem jamallaqdiem added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 11, 2026
@SlideGauge SlideGauge added the Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. label Mar 21, 2026

# TODO: Write a command to output the contents of the helper-1.txt file inside the helper-files directory to the terminal.
# The output of this command should be "Once upon a time...".
cat helper-files/helper-1.txt No newline at end of file

Choose a reason for hiding this comment

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

what will happen if we execute this script from the cat folder? (hint, check where's helper-files folder is located relative to cat folder)

Copy link
Author

Choose a reason for hiding this comment

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

Hi thanks for taking time to review my work, if the script get executed form cat folder will not found the helper-files directory.
As I double checked, the README ask that the script running inside the directory they're saved in. I will address the matter.


# TODO: Write a command to output every line in dialogue.txt that does not contain the word "Hello" (regardless of case).
# The output should contain 10 lines.
grep -vi "hello" dialogue.txt | wc -l No newline at end of file

Choose a reason for hiding this comment

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

Could you double check what this script outputs? The request is to output lines of dialogue satisfying some criteria

Copy link
Author

Choose a reason for hiding this comment

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

Hi thanks for taking time to review my work, I just addressed the matter, I should stop at the output, without adding the wc that output the total counts

# TODO: Write a command to output just the names of each player along with the total of adding all of that player's scores.
# Your output should contain 6 lines, each with one word and one number on it.
# The first line should be "Ahmed 15". The second line should be "Basia 37"
awk '{ total = 0; for (i = 2; i <= NF; i++) total += $i; print $1, total }' scores-table.txt No newline at end of file

Choose a reason for hiding this comment

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

do we indeed want to start summation from 2 item?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for taking time to review my work, no we need to start from i =3, My calculation was based that awk start at index 0, however it start at 1.


# TODO: Write a command to output the name of every `.txt` file in this directory which contains a line of dialogue said by the Doctor.
# The output should contain two filenames.
grep -l 'Doctor' *.txt No newline at end of file

Choose a reason for hiding this comment

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

What will happen if patient says a line with a word "Doctor"?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for taking time to review my work, you're right it will show that line as well, to avoid that I will add the colon and the regex anchor to tell the command to get only the first thing on the line.

# TODO: Write a command which lists the files in the child-directory directory, one per line, sorted so that the most recently modified file is first.
# The output should be a list of names in this order, one per line: helper-3.txt, helper-1.txt, helper-2.txt.

ls -1t -1 ./child-directory

Choose a reason for hiding this comment

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

do we need -1 flag twice?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for taking time to review my work, there is no need to keep the -1 flag twice, I forgot to delete the second one, thanks.

@SlideGauge
Copy link

Good direction, I placed some comments, please fix them. Also, could you doublecheck new lines in the end of scripts please?

@SlideGauge SlideGauge 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. Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Mar 21, 2026
@jamallaqdiem
Copy link
Author

Good direction, I placed some comments, please fix them. Also, could you doublecheck new lines in the end of scripts please?

I made sure in the setting VsCode , that it will add a new line automatically at save.

@jamallaqdiem jamallaqdiem added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants