carapace-sh / carapace-jq / 29285705816

69%

master: 69%

Build Type

Pull #5

github

Committed by rsteube-bot

Commit Message

Fix builtin completion suffixes to match jq calling conventions

Zero-arity builtins (length, sort, add, etc.) are called without
parentheses in jq, so they no longer get a `()` suffix. Builtins with
arguments (map, select, first(expr), etc.) now use NoSpace() so the
shell doesn't insert a trailing space after the open paren. Functions
with both 0-arity and 1-arity forms (first, last, any, all, flatten,
recurse, combinations, error) appear in both lists. Also fixed
miscategorized builtins: arg-requiring functions (getpath, path, del,
delpaths, pick) moved out of zero-arg list, and 0-arity-only functions
(floor, sqrt, abs, now, date functions, etc.) moved out of with-args
list. Removed duplicate ltrimstr entry.

Assisted-by: Crush:glm-5.2

Pull Request Pull Request #5: Fix builtin completion suffixes to match jq calling conventions

Coverage Stats

34 of 0 new or added lines in 0 files covered. (NaN%)

5 existing lines in 1 file now uncovered.

2817 of 4070 relevant lines covered (69.21%)

0.78 hits per line