Tomáš Bajer
Tomáš Bajer
@thrix Hello, I had that planned for this weekend, however I see Lukas's proposed changes in the draft and it seems they would would kinda overlap. Should I proceed or...
Hi @falconizmi, any update on this issue? Is it going to make it into 1.46?
I've found a typo in `tests/discover/adjust-tests.sh` so I've fixed it in this PR if that's okay. Also `mypy` found unused import of `subprocess` in `steps/discover/fmf.py` and removed it.
Addressed coderabbitai's comment in https://github.com/teemtee/tmt/pull/3701/commits/5d71903e89368b48ba0c5228dc5f8e0d9f3a6f1c @coderabbitai full review
@thrix I have improved the summary to describe the refactoring changes. I welcome any other improvement idea.
Let's go this way then @happz
All changes seem reasonable to me. I have one question though: removing the `# noqa: S320` in tmt/steps/report/junit.py will now make the linter complain about vulnerability of XML attacks, right?...