Skip to content

London | 26-Mar-SDC | Onur Atas | Sprint 2| Shell pipelines#487

Open
onurat wants to merge 1 commit into
CodeYourFuture:mainfrom
onurat:Shell-pipelines
Open

London | 26-Mar-SDC | Onur Atas | Sprint 2| Shell pipelines#487
onurat wants to merge 1 commit into
CodeYourFuture:mainfrom
onurat:Shell-pipelines

Conversation

@onurat
Copy link
Copy Markdown

@onurat onurat commented Apr 10, 2026

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

Solved shell pipeline exercises using grep, sort, uniq, and tr

@onurat onurat added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Apr 10, 2026
@LonMcGregor LonMcGregor added the Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. label May 13, 2026
Copy link
Copy Markdown
Contributor

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

Good work on this task, just one file to take a look at

# The input for this script is the events.txt file.
# TODO: Write a command to show how many times anyone has entered and exited.
# It should be clear from your script's output that there have been 5 Entry events and 4 Exit events.
sort events.txt | uniq -c No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you find any way to make it more obvious that there are 5 Entry and 4 Exit?

It should be possible to have this be the output, so the user doesn't have to add the numbers up as they are reading it.

@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. Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels May 13, 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