Skip to content

London | 26-SDC-March | Zobeir Rigi| Sprint 2 | jq#465

Open
Zobeir-Rigi wants to merge 6 commits intoCodeYourFuture:mainfrom
Zobeir-Rigi:jq
Open

London | 26-SDC-March | Zobeir Rigi| Sprint 2 | jq#465
Zobeir-Rigi wants to merge 6 commits intoCodeYourFuture:mainfrom
Zobeir-Rigi:jq

Conversation

@Zobeir-Rigi
Copy link
Copy Markdown

@Zobeir-Rigi Zobeir-Rigi commented Apr 4, 2026

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

Jq practice scripts for learning data extraction and filtering.



@Zobeir-Rigi Zobeir-Rigi added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Apr 4, 2026
Copy link
Copy Markdown

@cjyuan cjyuan left a comment

Choose a reason for hiding this comment

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

Solution looks good. I just have one question.

Comment thread jq/script-01.sh
# The input for this script is the person.json file.
# TODO: Write a command to output the name of the person.
# Your output should be exactly the string "Selma", but should not contain any quote characters.
jq -r ."name" person.json 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.

-Indentation is off

  • Why place double quotes around name?

@cjyuan cjyuan 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 9, 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