Skip to content

Fix issue #41: repo.is_dirty() on empty repository with stashed files #53

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
Closed
wants to merge 1 commit into from
Closed

Fix issue #41: repo.is_dirty() on empty repository with stashed files #53

wants to merge 1 commit into from

Conversation

fcelda
Copy link
Contributor

@fcelda fcelda commented Mar 21, 2012

No description provided.

@Byron
Copy link
Member

Byron commented Jun 7, 2012

Thanks for the fix.

However, without a test I will not be able to merge it in.
Please use nosetest to run the existing tests. You could add a new one which verifies your fix.

Thank you
@fcelda
Copy link
Contributor Author

fcelda commented Jun 7, 2012

The problem was that the tests didn't run on my machine, otherwise I would add the tests as well. Take a look at my previous comment on this: #41 (comment)

I do not remember what was the problem, I have never worked with nose before. But let me take a look again. ;-)
@Byron
Copy link
Member

Byron commented Nov 19, 2014

Thank you !
I believe your fix is already implemented by now.
@Byron Byron closed this Nov 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@fcelda @Byron