Glasgow | Prati Amalden | Individual shell tools | week 1#73
Glasgow | Prati Amalden | Individual shell tools | week 1#73PratiAmalden wants to merge 9 commits intoCodeYourFuture:mainfrom
Conversation
PratiAmalden
added
the
Needs Review
LonMcGregor
added
the
Review in progress
LonMcGregor
left a comment
LonMcGregor
left a comment
There was a problem hiding this comment.
Good work on these tasks - I just have two small followup comments
| # 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 |
There was a problem hiding this comment.
What will happen if the player makes many more attempts - will the command still work then?
There was a problem hiding this comment.
No it won't work, I fixed it.
There was a problem hiding this comment.
You don't need to commit .DS_Store files - these won't make sense if someone tries to pull your code onto a non-macos system
LonMcGregor
added
Reviewed
PratiAmalden
added
Complete
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s |
1 similar comment
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s |
|
Great work - you have completed this sprint task now. |
3 participants
Learners, PR Template
Self checklist
Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.