Do not initialize intercepts when loading from interactive shell

[firstone]

stdin/err intercepts should not be initialized when loading interactive shell. Otherwise, you cannot load module and issue help command, or just about do anything else in the shell, since output is intercepted.

I'd go a step further and never do it implicitly, instead doing it in Controller class but that might cause some incompatibility.

Rest of the changes are pep8 related minor fixes. They didn't break anything but created lots of linter noise.

[Goose66]

When will someone run a Polyglot nodeserver from the shell?

[firstone]

Most likely never, which is why it doesn't make sense to initialize logging / intercepts when loaded from it. This is the point of the change.

If you meant to ask why use this module in interactive shell to start with, the answer is - it's usually much easier to start/test initial development and the fastest way to get documentation without having to lookup project on github and diving into code or looking for other documentation.

[exking]

Looks good to me, definitely would make PyCharm happy