Remove automodule when running spellcheck.

[dHannasch]

Also changes the name of the environment variable so it'll be easier to see where it comes from next time someone looks at this code.

This is another way to silence that

WARNING: autodoc: failed to import module 'sampleproject'; the following exception was raised:
No module named 'sampleproject'

It is not necessary to do anything with the testsetup that imports the module, because unlike autodoc, doctest already automatically doesn't do anything if you don't ask for it; the testsetup becomes inert and therefore no import occurs.

[ionelmc]

But having 'source-read' handler just to silence warnings seems wrong in two ways:

• it's just a harmless warning
• it's doubling down on something wrong, why run those extensions anyway

Anyway, my proposal is to just remove the spell env. I've just tried it now on hunter it's horrendously cumbersome and broken - I have to whitelist names and things like docstring, backend or subclass. It's pointless really! <= @joaomcteixeira

[joaomcteixeira]

I have to whitelist names and things like docstring, backend or subclass. It's pointless really!

That's true for sure. hehe, I have no objections, go head and remove spellcheck altogether, definitively seems better than going around with complicated fixes just to try to maintain something that, as you said, is harmless and unmaintained.

Truth is that once one understands tox workflow it is easy to add additional checks, and the main template should have only the most general and consistent approaches.