🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко)

Results 3657 comments of 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко)

FWIW here's my playground: https://github.com/webknjaz/ansible-gentoo-laptop/pull/4/checks?check_run_id=35656446 And the source code: https://github.com/webknjaz/ansiwatch-bot

Yeah, I'm also not sure whether that solution is a good fit. OTOH I'd probably prefer just one switch to emit everything as JSON or similar, maybe JSONL. The problem...

Hm.. This looks similar to #74

Hm.. it's curious that some old versions (https://pypi.org/project/ansible-review/0.9.0/#description) have wheels and description published (misrendered markdown though)

Yep, working on submitting a similar issue to `ansible-lint` now.

Hey @nedbat, could you check this PR in the context of your report #20?

> Commenting the `my_fixture` out in `test_thing.py` makes pylint-pytest pass but executing tests fails to `my_fixture` not being found. This is weird and sounds like a bug in pytest itself....

When you have a fixture that is local to the test module, not global for the whole test session. Also, you may override the global fixtures with some values specific...

> e2e_test.py:7:20: W0613: Unused argument 'virtual_site' (unused-argument) This sounds reasonable. If you're not going to use the fixture's return value, just apply it via `@pytest.mark.usefixtures('virtual_site')`. Another pytest linting plugin (but...