This is in fact required for the pytest plugin loading mecanism to work
without hijacking the loading of swh packages.
See https://github.com/pytest-dev/pytest/issues/2042 for more details.

make mypy 0.730 pass cleanly again

None is the default value, and sub-classes could possible not redefine it to be
an empty list. Hence, this change provides a sane default for downstream users.

to prevent erratic errors when using heavily multithreaded code.

`from aiohttp import Response` raises an `ImportError`.

Otherwise exceptions are silently ignored, until the transaction finishes.
And when the transaction fails, the returned error message is unhelpful.

This commit makes copy_to return the actual error message.

For the content negotiation to work, we expect handler functions to
return an aiohttp_utils.Response object instead of an aiohttp.Response one
so we can handle the serialization in the "main" middleware (error_middleware,
which should probably be renamed).

swh.core.api.asynchronous.encode_data_server() can still be used; it will
produce a correct Response object if negotitation is in use.

It should still work with server handler implementations that do
produce aiohttp.Response objects, but without the content negotitation
feature available then.

This adds a dependency on the aiohttp_utils python package.

the deprecated decorator for classes does not return a new class, it only
inject a modified __new__ method, so we indeed need a new class object
in this case of class renaming.

The list of registered subcommands depends on the user's dev environment.

that allows to initialize the database for a given swh package with db
connection credentials/dsn read from a standard swh config file.

Also add a couple of tests (only tests click group and command exist).

Closes T1896.

and document it briefly in the README file.

This comes with a few checkers:
- flake8
- trailing-whitespace
- check-json
- check-yaml
- codespell

Also include commented checkers:
- black: disabled since we are not ready for this yet,
- pydocstyle: disabled since we have way too many pydoctyle errors for now...

See https://pre-commit.com/ for more details.

Related to T1881.

No need for it being a dynamically defined function, plus it makes pydocstyle
very unhappy (probably a bug in pydoctyle 4.0.0 but...).

Also rename it as _safemembers() for consistency sake.

use a newly created event loop to consume events in test_timed_coroutine.
ome reason, executing this later test in the same test session as
swh/core/api/tests/test_async.py fail because the main event loop seems
to be closed by the test.

json.loads() expect a str, not a bytes.

Note that test_post_struct_json is failing with current implementation,
the fix comes with the next revision.

- SWHRemoteAPI -> RPCClient
- SWHServerAPIApp -> RPCServerApp

to get rid of the SWH prefix and give them more consistent names, especially
the swh.api.asynchonous.SWHRemoteAPI class which is very confusing.

Deprecate old class names using the Deprecate python module.

by default:
- increase the connection pool size and
- set retry to 3 (number of retries each connection should attempt).

These config params can be set directly from config files for 'remote'
api clients. For example, to specify these in an objstorage client
config file:

  objstorage:
    cls: remote
    args:
      url: swh-objstorage:5003
      max_retries: 5
      pool_connections: 100

this is needed to prevent the command to depend on having pg's createdb
tool installed on the machine running the 'swh db-init' command.

In a normal deployment setup, this pg tool is not expected to be present
on every machine on which one want to be able to use this 'swh db-init'
command.

we want the root logger configured there, not the cli one.

to make sure each extra setup dependencies are ok.
This requires to move test files in its related subpackage.