pytest-dev / pytest

The pytest framework makes it easy to write small tests, yet scales to support complex functional testing
https://pytest.org
MIT License
11.95k stars 2.66k forks source link

details impact of specifying CI or BUILD_NUMBER #12578

Closed MarcBresson closed 3 months ago

MarcBresson commented 3 months ago

I also added the mention in pytest -h. Closes https://github.com/pytest-dev/pytest/issues/12577.

FYI, the doc of 8.x is not very well referenced, and when I was looking for this env var, I always was on the doc of version 7.x

nicoddemus commented 3 months ago

FYI, the doc of 8.x is not very well referenced, and when I was looking for this env var, I always was on the doc of version 7.x

Indeed, we have made steps towards improving this recently.

MarcBresson commented 3 months ago

the output of pytest -h in the reference doc seemed outdated. Maybe there could be a CI pipeline to check that somewhere ?

nicoddemus commented 3 months ago

the output of pytest -h in the reference doc seemed outdated. Maybe there could be a CI pipeline to check that somewhere ?

Do not worry about it, we update it at release time.

MarcBresson commented 3 months ago

should I drop my commit then ?

nicoddemus commented 3 months ago

Ahh no harm keeping it. :+1: :grin:

patchback[bot] commented 3 months ago

Backport to 8.2.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.2.x/6933bef0b0f193a32a3716a72a26e7184542376b/pr-12578

Backported as https://github.com/pytest-dev/pytest/pull/12587

🤖 @patchback I'm built with octomachinery and my source is open — https://github.com/sanitizers/patchback-github-app.