Issue 270

[widdowquinn]

Fixes #270

The origin of the warning was the rev: key for black. This was mutable (stable) but black doesn't support this, and so we have to pin the .pre-commit-config.yaml file to a specific black version.

In making the change the pinned dependency for blacken-docs in the same file was causing an issue, so I unpinned that.

Finally, I added blacken-docs to the requirements-dev.txt file.

Type of change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality not to work as expected)
• This change requires a documentation update

Action Checklist

• Work on a single issue/concept (if there are multiple separate issues to address, please use a separate pull request for each)
• Fork the pyani repository under your own account (please allow write access for repository maintainers)
• Set up an appropriate development environment (please see CONTRIBUTING.md)
• Create a new branch with a short, descriptive name
• Work on this branch
  • style guidelines have been followed
  • new code is commented, especially in hard-to-understand areas
  • corresponding changes to documentation have been made
  • tests for the change have been added that demonstrate the fix or feature works
• Test locally with pytest -v non-passing code will not be merged
• Rebase against origin/master
• Check changes with flake8 and black before submission
• Commit branch
• Submit pull request, describing the content and intent of the pull request
• Request a code review
• Continue the discussion at Pull requests section in the pyani repository

[codecov]

Codecov Report

Merging #271 (2ddcf26) into master (6d665d2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #271   +/-   ##
=======================================
  Coverage   75.82%   75.82%           
=======================================
  Files          52       52           
  Lines        3239     3239           
=======================================
  Hits         2456     2456           
  Misses        783      783           

[baileythegreen]

I've tested this in both the fastANI branch, and by pulling issue_270 and making a commit there. It seems to solve the problem.

[widdowquinn]

Thanks @baileythegreen - merging, now.