Closed brendrach closed 3 years ago
This failed the linter. You can run black
locally to see what the problem is
Code and test look good, thanks. I don't mind having this (which is rather self-contained) in enterprise core. Once the tests pass, this will be OK to merge.
Do we still need this in enterprise core before the feature freeze tomorrow?
We'll make a new PR to remove BWM functionality from enterprise once it's settled in enterprise_extensions.