Was a little nervous about changing some code in our new test framework.
It wasn’t necessary, but was was a bit nicer. Mostly cosmetic, much more pythonic. Added a bit of readability/privacy, but it’s unlikely to be used much in new places.
I figured I’d just do the work and test it, stick it on a merge request and if it wasn’t approved, move on.
One of the framework engineers came over in person yesterday to say he liked it. :)
Add comment