Cape Town| 25-SDC-July | Faith Muzondo | Sprint 2| Shell pipelines #119
Cape Town| 25-SDC-July | Faith Muzondo | Sprint 2| Shell pipelines #119Faithy4444 wants to merge 5 commits intoCodeYourFuture:mainfrom
Conversation
Faithy4444
added
the
Needs Review
LonMcGregor
left a comment
LonMcGregor
left a comment
There was a problem hiding this comment.
Hi,
- Remember to update the default PR text when submitting on github.
- You committed files that were not part of this PR - please be careful of that. Can you remove these from the "changed files" please?
I have left a comment in one file for you to look at as well
shell-pipelines/ls-grep/script-04.sh
Outdated
| # TODO: Write a command to count the number of 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 be the number 7. | ||
|
|
||
| ls sample-files | grep "^[A-Z][^A-Z]*$" |wc -l No newline at end of file |
There was a problem hiding this comment.
Do you know how to complete this using only lsand grep?
LonMcGregor
added
Reviewed
|
Good work |
LonMcGregor
added
Complete
|
Your PR description contained template fields which weren't filled in. Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed. If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
1 similar comment
|
Your PR description contained template fields which weren't filled in. Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed. If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
2 participants
Learners, PR Template
Self checklist
Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.