Conversation
Nikoleta-v3
left a comment
There was a problem hiding this comment.
Looks great 👍🏻 Thank you @marcharper
|
@drvinceknight PTAL |
|
It's failing on We have not pinned a version of Two solutions:
I'm agnostic: whatever you prefer Marc :) |
|
I'm fine either way. We can pin black in another PR (do you know the version?) but if we're going to rerun black on the entire codebase, let's merge this first. |
|
Sounds good to me. Checking So as far as I can tell Should work. |
|
Merging, opened #1412 to pin black version. |
Tests pass locally, seems black wants to reformat a large number of files. I suggest doing that in a separate PR afterward.