-
Notifications
You must be signed in to change notification settings - Fork 172
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
Release 1.1.6 #295
Release 1.1.6 #295
Conversation
This adds a [pre-commit](https://pre-commit.com)-hook for running Prospector against all changed Python files of a git commit.
…encies in list of used libraries
Add a pre-commit hook for running Prospector
Adding pylint plugins back to fix --uses and --autodetect
…c separator rather than OS specific path separator
@chocoelho FYI I also changed the threshold for failure on coveralls to a change of 1% so we don't get "failed" PRs due to a 0.04% change in coverage... |
More generic path separator for --output-format output files
splitted_value = value.split(pathsep) | ||
output_format, output_targets = splitted_value[0], splitted_value[1:] | ||
parsed = re.split(r'[;:]', value) | ||
output_format, output_targets = parsed[0], parsed[1:] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this raises another issue least if split by ':'
is used on windows and this is still supposed to work with absolute paths as -o xunit:C:\test_results\test.xml
will behave unexpectedly i think (not sure how this is used elsewhere)
Two small fixes and also bringing develop and master branch into sync
@chocoelho No big changes here, one small fix and just want to clean up the master/develop differences. I'll merge this myself if you don't have any complaints in the next few days.