Skip to content

London | 26-March-SDC | Ammad Ur Rehman | Sprint 1 | Individual Shell Tools#420

Open
anosidium wants to merge 6 commits intoCodeYourFuture:mainfrom
anosidium:individual-shell-tools
Open

London | 26-March-SDC | Ammad Ur Rehman | Sprint 1 | Individual Shell Tools#420
anosidium wants to merge 6 commits intoCodeYourFuture:mainfrom
anosidium:individual-shell-tools

Conversation

@anosidium
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

Done all the required exercise with some stretch exercises completed

Questions

I find reading the manual pages of the commands confusing and looking them up on the internet isn’t much better either.

@anosidium anosidium added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 23, 2026
Copy link
Copy Markdown

@SlideGauge SlideGauge left a comment

Choose a reason for hiding this comment

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

Could you adjust the settings of your editor and adjust all the scripts, they are missing the trailing new line

# Your output should contain 3 lines, each with one word and one number on it.
# The first line should be "Ahmed 4".

awk '{print $1, $5}' scores-table.txt No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  1. Will it output London-only players?
  2. Do all the players have 5 columns?

# TODO: Write a command to output every line in dialogue.txt said by the Doctor.
# The output should contain 6 lines.

grep '^Doctor' dialogue.txt No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What will happen if have a dialogue line from a character like this:
DoctorlikeButNotADoctor: la la la

# 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
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Will this pattern ignore something like this:
Patient: Doctor, what should I do?

# Line 3 should be "It contains many lines, and there are some things you may want to do with each of them.".
# Line 11 should be "We also should remember, when we go shopping, to get 4 items: oranges, cheese, bread, olives.".

sed -E 's/,([^ ])/\, \1/g' input.txt No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do we need to escape comma via ","?

@@ -16,8 +16,12 @@ echo "First exercise (sorted newest to oldest):"
# 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.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Does the approach output one-per-line in this script?

# I was tempted to take a bite of it.
# But this seemed like a bad idea...

cat ../helper-files/helper-1.txt ../helper-files/helper-2.txt ../helper-files/helper-3.txt No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do we need to list all the files manually if the task asks tou output all the files inside the directory?

@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. labels Apr 17, 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