Correct index.lock file deletion and other fixes & formatting#52
Correct index.lock file deletion and other fixes & formatting#52sebdah merged 3 commits intosebdah:masterfrom
Conversation
|
@sebdah Can we please get this rolled out. This is important and urgent. I don't know how to mark this a showstopper. |
sebdah
left a comment
There was a problem hiding this comment.
Thanks for the submission @sandipagarwal. I have added a couple of comments on your code.
Sorry for the late response to this, I have been traveling. I'll get this merged and released as soon as the comments are addressed.
git-pylint-commit-hook
Outdated
|
|
||
| import argparse | ||
| import os | ||
| # import os |
There was a problem hiding this comment.
Remove instead of commenting out.
git-pylint-commit-hook
Outdated
| if os.path.exists('.git/index.lock'): | ||
| os.remove('.git/index.lock') | ||
| # if os.path.exists('.git/index.lock'): | ||
| # os.remove('.git/index.lock') |
There was a problem hiding this comment.
Remove instead of commenting out.
git-pylint-commit-hook
Outdated
| @@ -72,8 +72,8 @@ def main(): | |||
| sys.exit(0) | |||
|
|
|||
| # Remove the index.lock when it's exists | |||
There was a problem hiding this comment.
This comment should also be removed.
| try: | ||
| if _is_python_file(filename) and \ | ||
| not _is_ignored(filename, ignored_files): | ||
| if _is_python_file(filename) and not _is_ignored(filename, ignored_files): |
There was a problem hiding this comment.
Why is this done? It seems like this line exceeds 80 chars. Please revert.
There was a problem hiding this comment.
My bad. Editor was configured so.
Done.
|
Thanks @sebdah. I have fixed all review comments from you. |
|
This has now been released in version 2.2.1. Thanks again @sandipagarwal for the submission! |
|
Thanks @sebdah. :) |
The following have been fixed.