Update code so that the `warn_return_any` check passes.

Add a custom argparse help formatter that overrides the output
format to list items vertically.

The formatter is derived from argparse.HelpFormatter with minimal changes.

Co-authored-by: John Villalovos <john@sodarock.com>
Co-authored-by: Nejc Habjan <nejc.habjan@siemens.com>

chore: enable more pylint checks

Rename the variables `whaction` and `action` to `resource_action` to
improve code-readability.

Enable the pylint check "redefined-outer-name" and fix the errors
detected.

Enable the pylint checks:
  * unnecessary-pass
  * unspecified-encoding

Update code to resolve errors found

Naming a variable `what` makes it difficult to understand what it is
used for.

Rename it to `gitlab_resource` as that is what is being stored.

The Gitlab documentation talks about them being resources:
https://docs.gitlab.com/ee/api/api_resources.html

This will improve code readability.

Previously in commit 233b79ed442aac66faf9eb4b0087ea126d6dffc5 I had
used the `name` argument for `ModuleNotFoundError()`. This basically
is the equivalent of not passing any message to
`ModuleNotFoundError()`. So when the exception was raised it wasn't
very helpful.

Correct that and add a unit-test that shows we get the message we
expect.

fix(cli): change default `allow_abbrev` value to fix argument collision