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

inline compatible jq args to support --in-place #15

Merged
merged 9 commits into from
Sep 18, 2023
Merged

inline compatible jq args to support --in-place #15

merged 9 commits into from
Sep 18, 2023

Conversation

clux
Copy link
Owner

@clux clux commented Sep 17, 2023

  • preliminary work fixing trailing vararg quirks by inlining common jq flags inside our args
  • add explicit file arg to help this and fix awkward stdin detection
  • rename -i to --input (people can alias tq=yq --input=toml as a one time thing where it's more readable with the full arg)
  • add new -i / --in-place that requires file arg

Signed-off-by: clux <sszynrae@gmail.com>
@clux clux linked an issue Sep 17, 2023 that may be closed by this pull request
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux linked an issue Sep 17, 2023 that may be closed by this pull request
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux changed the title inplace wip inline compatible jq args to support --in-place Sep 17, 2023
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux marked this pull request as ready for review September 18, 2023 18:50
@clux clux merged commit 23ccaf4 into main Sep 18, 2023
13 checks passed
@clux clux deleted the yq-inplace branch September 18, 2023 18:55
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.

allow in-place editing stdin usage detection broken on ci
1 participant