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.

Sign up for GitHub

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

Jump to bottom

`jsonp`: use `print!` instead of `println!` & add `House.csv` + tests #1897

Merged

jqnatividad merged 3 commits into master from jsonp-newline

Jun 19, 2024

Collaborator

rzmk commented Jun 19, 2024 •

edited

Loading

Uses print! when providing output for qsv jsonp. Also adds a new test resource House.csv and tests as discussed in #1892.

rzmk added 3 commits

June 19, 2024 06:51


          jsonp: use print! instead of println! and add House.csv + tests

a5f1fbf


          tests: refactor test name

61b408b


          tests: use enum & diff -k to pass all jsonp tests

750bb8e

jqnatividad merged commit 9efb7cd into master

13 of 15 checks passed

rzmk deleted the jsonp-newline branch

June 19, 2024 12:06

jqnatividad mentioned this pull request

CSV -> JSON -> CSV conversion diff and loss of double quotes #1892

Closed

BrewTestBot mentioned this pull request

qsv 0.129.0 Homebrew/homebrew-core#177318

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment