Skip to content

London | SDC-Nov-2025| Ikenna Agulobi | Sprint 1 | Individual Shell Tools#240

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

London | SDC-Nov-2025| Ikenna Agulobi | Sprint 1 | Individual Shell Tools#240
ike-agu wants to merge 6 commits intoCodeYourFuture:mainfrom
ike-agu:individual-shell-tools

Conversation

@ike-agu
Copy link

@ike-agu ike-agu commented Nov 20, 2025

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

  • ls: Add ls command flags to list files as required in exercise
  • cat: Add cat commands to output file contents as required in exercise
  • wc: use wc commands to count words, lines and character in files
  • grep: Add grep command to count Doctor lines across dialogue files
  • sed: use sed commands to replace, modify and insert texts

@ike-agu ike-agu added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Nov 20, 2025
Copy link

@LonMcGregor LonMcGregor left a comment

Choose a reason for hiding this comment

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

This looks good to me, see my comment in one of the files for some extra advice

Choose a reason for hiding this comment

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

Correct, but remember you can combine some single-character options into one, rather than listing them separately

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for reviewing my PR and for the feedback.

@LonMcGregor LonMcGregor added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Nov 25, 2025
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. 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants