Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change tutorial to highlight filter part of query versus output part of query #1460

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

stagrlee
Copy link
Contributor

@stagrlee stagrlee commented Aug 8, 2017

Intent of this change is to let the reader who is totally unfamiliar with the JQ syntax see how the input JSON is filtered during a query by the right side of the of the syntax and the output is specified by the left side. In the current tutorial, the search filter is

.commit.message

and the output is

message:

The word message is used in both the search side and the output side and might confuse a new user to try to search JSON input with the left side. (It has happened to me a few times after leaving JQ coding for a few months). With this patch, search remains the same but "message" turns into "comment" and "committer" turns into "contributor" showing the reader that the left side is clearly output and the right side is the search.

The configure.ac change is to keep autoconf syntax highlighters from getting confused.

@coveralls
Copy link

coveralls commented Aug 8, 2017

Coverage Status

Coverage increased (+0.4%) to 84.98% when pulling 21c3470 on stagrlee:master into 341a5fc on stedolan:master.

@stagrlee
Copy link
Contributor Author

stagrlee commented Aug 9, 2017

Travis-ci error is the same error seen on master so I believe the problem is upstream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants