pydoclint: Disable --skip-checking-short-docstrings by default

[llucax]

What's needed?

We are currently enabling the --skip-checking-short-docstrings option to avoid having redundant documentation for simple methods and also properties.

As a downside, some documentation might be missing accidentally when people write a short docstring, but they really don't want to skip all the details, specially in cases where there are extra arguments.

Google coding style allows for this omission but only for Returns:

It may also be omitted if the docstring starts with Returns or Yields (e.g. """Returns row from Bigtable as a tuple of strings.""") and the opening sentence is sufficient to describe the return value.

This makes more sense and should be much safer.

Proposed solution

Disable --skip-checking-short-docstrings, but that would leave us with a lot of redundant documentation again.

Ideally we should have much more control about how much info can be missing, but this is not yet possible with pydoclint. So we might need to wait until these get implemented (or try to send PRs ourselves):

• Google style: Allow omitting only the Returns/Yields if summary starts with Return(s)/Yield(s) jsh9/pydoclint#110
• Google style: Allow documenting @propertys as a member jsh9/pydoclint#111
• Allow more control over--skip-checking-short-docstrings jsh9/pydoclint#112

[llucax]

@shsms FYI