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

add cat --csv --no-header option #31

Merged
merged 6 commits into from
Aug 2, 2022

Conversation

jeffbski-rga
Copy link
Contributor

pqrs cat --csv-no-header parquetFile will output a csv without the header line

Jeffrey Barczewski added 2 commits August 1, 2022 17:50
`pqrs cat --csv-no-header parquetFile` will output a csv without the headeradd cat --csv-no-header option
`pqrs cat --csv-no-header parquetFile` will output a csv without the header line
README.md Outdated Show resolved Hide resolved
Jeffrey Barczewski and others added 3 commits August 2, 2022 13:57
Instead of combining the argument as `--csv-no-header`, make `--no-header` an additional flag for `--csv`.

For example: `pqrs cat --csv --no-header parquetFile`
@jeffbski-rga
Copy link
Contributor Author

It looks like my editor put in a few markdown syntax changes by accident. I can revert those if necessary.

@jeffbski-rga
Copy link
Contributor Author

ok. I removed those markdown changes

@manojkarthick manojkarthick merged commit c1ab1ed into manojkarthick:master Aug 2, 2022
@manojkarthick
Copy link
Owner

Thanks a lot! I will release a new version with this change shortly.

@jeffbski-rga
Copy link
Contributor Author

Awesome! Thanks for the quick turn around.

@jeffbski-rga jeffbski-rga deleted the cat-csv-no-header branch August 2, 2022 19:29
@jeffbski-rga jeffbski-rga changed the title add cat --csv-no-header option add cat --csv -no-header option Aug 2, 2022
@jeffbski-rga jeffbski-rga changed the title add cat --csv -no-header option add cat --csv --no-header option Aug 2, 2022
@jeffbski-rga
Copy link
Contributor Author

@manojkarthick Will you be able to deploy a new release soon? I would love to reference your version to get this feature rather than building my own. Thanks in advance!

@manojkarthick
Copy link
Owner

@manojkarthick Will you be able to deploy a new release soon? I would love to reference your version to get this feature rather than building my own. Thanks in advance!

Sorry for the delay @jeffbski-rga - the Release workflow is broken and I've had time to fix it yet. I've published it on Cargo now, not sure that's how you plan to reference it

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.

2 participants