Cape Town| 25-SDC-July | Faith Muzondo | Sprint 2| jq Commands#114
Cape Town| 25-SDC-July | Faith Muzondo | Sprint 2| jq Commands#114Faithy4444 wants to merge 3 commits intoCodeYourFuture:mainfrom
Conversation
Faithy4444
added
the
Needs Review
illicitonion
changed the title
LonMcGregor
added
Review in progress
LonMcGregor
left a comment
LonMcGregor
left a comment
There was a problem hiding this comment.
Hi, I just have one comment for one task. Good work on all the others.
jq/script-02.sh
Outdated
| # The input for this script is the person.json file. | ||
| # TODO: Write a command to output the address of the person, all on one line, with a comma between each line. | ||
| # Your output should be exactly the string "35 Fashion Street, London, E1 6PX", but should not contain any quote characters. | ||
| jq -r '.address[0] + ", " + .address[1]+ ", "+ .address[2]' person.json No newline at end of file |
There was a problem hiding this comment.
Can you design a solution which works for any length of address - even with more than 3 lines?
LonMcGregor
added
Reviewed
|
Great 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.