Skip to content

GHA-delete-GNU-workflow-logs.sh: Add fallback to jaq - #9581

Merged
sylvestre merged 1 commit into
uutils:mainfrom
oech3:patch-4
Dec 7, 2025
Merged

sylvestre merged 1 commit into
uutils:mainfrom
oech3:patch-4

Conversation

@oech3

@oech3 oech3 commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Dec 6, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/tty/tty-eof is no longer failing!

@sylvestre

Copy link
Copy Markdown
Contributor

Sorry but why? :)

@oech3

oech3 commented Dec 6, 2025

Copy link
Copy Markdown
Contributor Author

Useful a bit. (Main reason is cleanup for .sh)

@sylvestre

Copy link
Copy Markdown
Contributor

Sorry but I don't understand "Useful a bit.":)

@oech3

oech3 commented Dec 6, 2025

Copy link
Copy Markdown
Contributor Author

Some distribution does not provide this command as drop-in.
How about supporting custom JQ variable instead if this looks odd?

@sylvestre

Copy link
Copy Markdown
Contributor

Ok , thanks
Maybe add this in the commit message

…t jaq for the case it is not installed as jq
@oech3 oech3 changed the title GHA-delete-GNU-workflow-logs.sh: Support jaq GHA-delete-GNU-workflow-logs.sh: Support fallback to jaq Dec 6, 2025
@oech3 oech3 changed the title GHA-delete-GNU-workflow-logs.sh: Support fallback to jaq GHA-delete-GNU-workflow-logs.sh: Add fallback to jaq Dec 6, 2025
@github-actions

github-actions Bot commented Dec 6, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@sylvestre
sylvestre merged commit 3528d10 into uutils:main Dec 7, 2025
129 checks passed
@oech3
oech3 deleted the patch-4 branch December 7, 2025 16:09
@xtqqczze

Copy link
Copy Markdown
Collaborator

By the way, the gh command has a --jq option that lets you run the query server-side, which can be more efficient:

  -q, --jq string                Query to select values from the response using jq syntax

sylvestre pushed a commit that referenced this pull request Sep 29, 2026
…d support jaq for the case it is not installed as jq (#9581)"

This reverts commit 3528d10.
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.

3 participants