remove git fetch from tox docs environment

[jakkdl]

It's a pain to have these when testing locally, they're unnecessary, and will prompt for ssh password (my ssh-agent env vars aren't passing through to tox).

Testing to see if this breaks CI

[webknjaz]

Looks like it's okay

[nicoddemus]

Thanks!

[RonnyPfannschmidt]

I vaguely recall that read the docs has a extra setting for this now

Let's crosscheck that just in case

[jakkdl]

I vaguely recall that read the docs has a extra setting for this now

Let's crosscheck that just in case

are you referring to the stuff in .readthedocs.yaml?

pytest/.readthedocs.yaml

Lines 24 to 26 in 1f1a10c

	post_checkout:
	- git fetch --unshallow || true
	- git fetch --tags || true

idk if that's needed, but it also doesn't bother me when developing locally so /shrug

[RonnyPfannschmidt]

It's needed and pretty much the replacement for the one you correctly removed

[patchback]

Backport to 8.3.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.3.x/de1a4885d6fd27e4189aba94b8922acc58364142/pr-13199

Backported as #13200

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

[webknjaz]

It's needed and pretty much the replacement for the one you correctly removed

@RonnyPfannschmidt do you remember why? It might not be. I think we dropped the dependency on Git from the Sphinx config in all envs to make cache work better on the sprint, right?

[RonnyPfannschmidt]

We needed this for correct setuptools_scm Metadata on rtd

[webknjaz]

@RonnyPfannschmidt I thought so too. But when I checked earlier today, I haven't found evidence of setuptools-scm being used there.. Am I missing something?

[RonnyPfannschmidt]

Install on rtd would fail due to shallow clones

[webknjaz]

@RonnyPfannschmidt are you sure? I think it was the case when there was setuptools-scm in play but not anymore.