Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix failing on existing Git comment char #48

Merged
merged 1 commit into from
Jan 19, 2018

Conversation

torwald-sergesson
Copy link
Contributor

As soon as _git function returns string if it was executed (not sh.RunningCommand) get_commentchar
functions fails with AttributeError (could not get exit_code).

@torwald-sergesson
Copy link
Contributor Author

I see one failing check but it fails only on 2.7 and this check is not related to my code.

As soon as _git function returns string if it was executed (not sh.RunningCommand) **get_commentchar**
functions fails with AttributeError (could not get exit_code).
@VladimirKasatkin
Copy link

Hi!
Will this pull request merged? I wanted to try gitlint on our team, but I do not want to use my own fork. Thanks!

@jorisroovers
Copy link
Owner

Not sure when this actually occurs, but the code itself can't hurt, so merging this.

FYI that I'm not immediately planning on rolling another release - so you'll need to install from git if you want your team to use the version with this fix.

@jorisroovers jorisroovers merged commit 8ff995b into jorisroovers:master Jan 19, 2018
@jorisroovers
Copy link
Owner

Thanks for your interest!

@jorisroovers
Copy link
Owner

Related issue: #70

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants