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 "exclude" command #206

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

lalaithion
Copy link

Issue #61 describes an idea for an exclude command. I needed the functionality of this command, so I added it. The code for it is largely based on the join command.

I also formatted the repository using cargo fmt.

@BurntSushi
Copy link
Owner

Please don't run rustfmt. I want to eventually switch to it, but doing so would make all extant PRs difficult to merge. Including rustfmt in this PR decreases its chances of me merging it.

Also added command to top level description.
@lalaithion
Copy link
Author

Undid rstfmt.

@rodion-k
Copy link

rodion-k commented Apr 12, 2020

I have tested it and find that rows are not excluded if they have empty fields. For example:
file1.csv

id,name
1,test1
2,test2
3,

file2.csv

id,name
1,test1
2,test2updated
3,

xsv exclude - file1.csv - file2.csv

id,name
2,test2updated
3,

Row 3 is not excluded.

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.

None yet

3 participants