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

Extended CSV Output #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Extended CSV Output #7

wants to merge 1 commit into from

Conversation

dotEvan
Copy link

@dotEvan dotEvan commented Apr 6, 2012

Added --csv option to the preview mode. This clears all other output to stdout except information about which branches are able to be deleted. In addition it supplies the date of the last commit and the committer name.

Sample:
$ ./bin/git-sweep preview --csv
command-line,2012-03-26,robmadole
develop,2012-03-26,robmadole
release-0.1.0,2012-03-21,robmadole
release-0.1.1,2012-03-28,robmadole

This can then be imported into something like Google Docs where the responsible parties can then ensure their branches are able to be deleted.

I also added a unit test, but because of the changing date and author of the output of each branch, just doing an assertion of exact string matches won't work that well. I've never used Python before, so not being familiar with, well, anything isn't lending itself to me finding an awesome way to test this. Hopefully someone else can provide a better way of being able to unit or, if you think counting commas will suffice for now, even better :)

…h name, last commit date and last commit author for each branch in CSV format. added unit test for verifying csv output.
@robmadole
Copy link
Collaborator

Very cool. Thank you for working on this.

If you can update this pull request to drop into develop instead of master I will merge this in and then maybe work on that test that you mentioned.

I think this link will get you to a screen that will let you edit this PR https://github.com/dotEvan/git-sweep/pull/new/arc90:master...dotEvan:feature/csv

@dotEvan
Copy link
Author

dotEvan commented Apr 23, 2012

Huh. Looks like that link just created a new pull request... see #9.

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