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

Simplified standard library; marked redundant filters as duplicated #426

Closed
wants to merge 1 commit into from
Closed

Simplified standard library; marked redundant filters as duplicated #426

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 17, 2014

See #418 for background discussion: quoting the gist of it here:

My suggestion is simple: functions that are identical save for an optional parameter (such as all functions that end with _by) could drop that prefix and become multi-arity functions


  • recurse takes over recurse_down and .., the former is deprecated.
  • min and max take over min_by and max_by, which are deprecated.
  • unique takes over unique_by, which is deprecated.
  • sort takes over sort_by, which is deprecated.
  • group replaces group_by, which is deprecated, for consistency.
  • paths gains an optional argument, so that leaf_paths is the same as paths(values). The former is deprecated.
  • range allows a one parameter invocation which assumes the range to start at zero.

"Drop the _by. It's cleaner."

@ghost
Copy link
Author

ghost commented Jun 17, 2014

s/duplicated/deprecated (oops! :$)

@nicowilliams
Copy link
Contributor

I rebased it, restored the missing recurse_down, massaged the manual a bit, and pushed it. Then I added some commits to get the manual to be a bit more consistent. Thanks!

@ghost
Copy link
Author

ghost commented Jun 17, 2014

Nice! Thank you. Also, sorry about recurse_down.

2014-06-18 1:23 GMT+02:00 Nico Williams notifications@github.com:

I rebased it, restored the missing recurse_down, massaged the manual a
bit, and pushed it. Then I added some commits to get the manual to be a bit
more consistent. Thanks!


Reply to this email directly or view it on GitHub
#426 (comment).

matthewfeickert added a commit to yadage/packtivity that referenced this pull request Sep 19, 2023
* The 'leaf_paths' filter has been deprecated for 9 years and was
  removed in jqlang v1.7 (jq v1.6) so use the 'paths(scalars)' filter.
   - c.f. jqlang/jq#2666
   - c.f. jqlang/jq#426
* Remove upper bound on jq version
   - Reverts #97
matthewfeickert added a commit to yadage/packtivity that referenced this pull request Sep 19, 2023
* The 'leaf_paths' filter has been deprecated for 9 years and was
  removed in jqlang v1.7 (jq v1.6) so use the 'paths(scalars)' filter.
   - c.f. jqlang/jq#2666
   - c.f. jqlang/jq#426
* Remove upper bound on jq version
   - Reverts #97
matthewfeickert added a commit to yadage/packtivity that referenced this pull request Sep 19, 2023
…ted (#101)

* Backport PR #99
* The 'leaf_paths' filter has been deprecated for 9 years and was removed in jqlang v1.7 (jq v1.6) so use the 'paths(scalars)' filter.
   - c.f. jqlang/jq#2666
   - c.f. jqlang/jq#426
* Remove upper bound on jq version
   - Reverts #97
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.

1 participant