Why this is moved?

Why this is moved?

Just for readability. now the register task and the task that uses the registered var are one below the other instead of having a bunch of unrelated tasks in the middle, it's easier to read what's going on imo. But I could remove that commit =)

No need to remove it, if it improves readability.

Otherwise looks good

:( I like the old style better

Thanks for these fixes!

:( I like the old style better

me too, but black does not. black from pip (19.10b0) does not like the old style. rpm black (19.0b0) likes both of them.

The other option is lock the requirement, or let vagrant fail on black tests