← Back to team overview

sts-sponsors team mailing list archive

Re: [Merge] ~adam-collard/maas:bump-black-to-23.1 into maas:master

 

Review: Approve

minor comment inline

Diff comments:

> diff --git a/src/maasserver/testing/testcase.py b/src/maasserver/testing/testcase.py
> index 143a288..530eaa0 100644
> --- a/src/maasserver/testing/testcase.py
> +++ b/src/maasserver/testing/testcase.py
> @@ -346,6 +347,7 @@ class UniqueViolationTestCase(
>          ought to tell us about it.
>  

should this one be blank?

>          """
> +
>          # Helper to switch the transaction to REPEATABLE READ.
>          def set_repeatable_read():
>              with connection.cursor() as cursor:


-- 
https://code.launchpad.net/~adam-collard/maas/+git/maas/+merge/437195
Your team MAAS Committers is subscribed to branch maas:master.



References