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

Typo: s/more/mode/ in --seq #1270

Merged
merged 1 commit into from
Jan 23, 2017
Merged

Typo: s/more/mode/ in --seq #1270

merged 1 commit into from
Jan 23, 2017

Conversation

timmc
Copy link
Contributor

@timmc timmc commented Nov 4, 2016

I'm not actually sure I understand that sentence, but it does appear to be a typo either way.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.333% when pulling e245f62 on timmc:patch-1 into 0b82185 on stedolan:master.

@eric-brechemier
Copy link
Contributor

@timmc If this sentence means that jq with the --seq option shall accept the output of jq used without the --seq option, it looks like a bug to me:

jq to jq --seq

$ jq -n '1,2,3' | jq --seq '.' | xxd # produces no output

The other forms work as expected:

jq --seq to jq --seq

$ jq --seq -n '1,2,3' | jq --seq '.' | xxd
0000000: 1e31 0a1e 320a 1e33 0a                   .1..2..3.

jq to jq

$ jq -n '1,2,3' | jq '.' | xxd
0000000: 310a 320a 330a                           1.2.3.

even

jq --seq to jq

$ jq --seq -n '1,2,3' | jq '.' | xxd
0000000: 310a 320a 330a                           1.2.3.

@nicowilliams nicowilliams merged commit f33af0b into jqlang:master Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants