Skip to content

London | 26-SDC-Mar | Jamal Laqdiem | Sprint 2 | shell pipelines exercises#367

Open
jamallaqdiem wants to merge 3 commits intoCodeYourFuture:mainfrom
jamallaqdiem:shell-pipelines
Open

London | 26-SDC-Mar | Jamal Laqdiem | Sprint 2 | shell pipelines exercises#367
jamallaqdiem wants to merge 3 commits intoCodeYourFuture:mainfrom
jamallaqdiem:shell-pipelines

Conversation

@jamallaqdiem
Copy link
Copy Markdown

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

Completed the exercises shell pipelines.

@jamallaqdiem jamallaqdiem added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 📅 Sprint 2 Assigned during Sprint 2 of this module labels Mar 13, 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? All the files are missing the trailing end of line.


# TODO: Write a command to output the names of the files in the sample-files directory whose name starts with an upper case letter and doesn't contain any other upper case letters.
# Your output should contain 7 files.
ls -1 sample-files | grep '^[A-Z][a-z]*$' 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.

The regex '^[A-Z][a-z]*$' captures "starts with uppercase, then only lowercase letters." But the task says "doesn't contain any other upper case letters" — it says nothing about restricting digits, dots, hyphens, etc. A filename like Foo.txt or File1 would NOT match your regex even though it satisfies the task requirement.

# Your output should be the number 7.
#!/bin/bash

ls -1 sample-files | grep '^[A-Z][a-z]*$' | wc -l 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.

Same regex concern as script-03

@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 May 2, 2026
@SlideGauge
Copy link
Copy Markdown

Good job, could you do a couple of fixes I pointed out?

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. 📅 Sprint 2 Assigned during Sprint 2 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants