search

cekit / behave-test-steps

MIT License
0 stars 22 forks source link

Fix duplicate log configuration #44

Closed rnc closed 1 year ago

rnc commented 1 year ago

Fixes https://github.com/cekit/cekit/issues/840

Also fix some minor warnings shown up through IntelliJ.

jmtd commented 1 year ago

the others seem to be generated from other bits of cekit, i.e. cli.py and are not problems with behave-test-steps

Scratch that; now I've made double-sure I'm referencing this branch for my tests those are gone too.

(FWIW: the --steps-url command for cekit can be tricksy. It clones the referenced repository, and checks out the v1 branch. So I have to make sure a local git clone of your fork has your PR branch merged into v1 and the remote then deleted so origin/v1 doesn't exist to be absolutely sure the right code is being run)