execute.c: support pick(last) #2779
Conversation
| jv_free(k); | ||
| k = jv_number(jv_array_length(jv_copy(t)) + idx); | ||
| } | ||
| } |
There was a problem hiding this comment.
This patch should be authored by @nicowilliams
There was a problem hiding this comment.
@emanuele6 - It was.
|
@pkoppstein please do nothing. I'll push my version of this with attribution to you. |
|
. |
|
Ok, @pkoppstein I've force-pushed to this PR. Please do not push to it again. Just tell me if it now has the contents that you wanted. If it does then I'll merge it after all the tests pass. |
|
@pkoppstein I've added the following wiki page for you: https://github.com/jqlang/jq/wiki/Guide-for-Contributors |
|
@nicowilliams wrote:
It's missing one minor comment, but that's OK. I see that "All checks have passed" so please merge it before things get complicated again. Thank you. |
OK, we're in the middle of trying to create a signed |
Support [0] | setpath([-1]; _)
Add tests for pick/1
Supersedes #2717