feat: Update packages and test settings (#55)

* chore: Extend gitignore

* chore: update packages

* chore: package update

* test: monkey patch for running the new mayjor version of pytest-asyncio

* test: Change docker container registry

* fix: explizit packages

* fix: remove gitignore configuration

* fix: pylint rollout of the code.

* remove .python-version

* fix: update dockerfile.

* fix: update poetry.lock

* fix: add  pylint.extensions.empty_comment

* fix: .gitiignore add default values

* fix: empty row at the end of the file

Author: atstaeff

.gitignore

@@ -1,6 +1,2 @@
-# Python folders
-__pycache__/
-
 # OS generated files and folders
 .DS_Store
-npm-debug.log

.pylintrc

@@ -13,7 +13,6 @@ load-plugins = pylint.extensions.bad_builtin,
 	pylint.extensions.docstyle,
 	pylint.extensions.dunder,
 	pylint.extensions.empty_comment,
-	pylint.extensions.emptystring,
 	pylint.extensions.eq_without_hash,
 	pylint.extensions.for_any_all,
 	pylint.extensions.magic_value,

eventsourcingdb_client_python/client.py

@@ -18,6 +18,11 @@
 from .handlers.write_events import Precondition, write_events
 
 
+# pylint: disable=R6007
+# Reason: This class explicitly specifies the return type as None
+# for better readability. Even though it is not necessary,
+# it makes the return type clear without needing to read any
+# documentation or code.
 class Client(AbstractBaseClient):
     def __init__(
         self,
@@ -38,7 +43,7 @@ def __init__(
     async def initialize(self) -> None:
         await self.__http_client.initialize()
 
-    async def close(self):
+    async def close(self) -> None:
         await self.__http_client.close()
 
     @property

eventsourcingdb_client_python/event/event.py

@@ -6,7 +6,11 @@
 
 Self = TypeVar("Self", bound="Event")
 
-
+# pylint: disable=R0917
+# Reason: This class is expected to have many parameters
+# due to its business context. Splitting it into smaller
+# methods would increase cognitive load and make the
+# code less readable.
 class Event(EventContext):
     def __init__(
         self,

eventsourcingdb_client_python/event/source.py

@@ -3,6 +3,11 @@
 from ..event.event_candidate import EventCandidate
 
 
+# pylint: disable=R0917
+# Reason: This class is expected to have many parameters
+# due to its business context. Splitting it into smaller
+# methods would increase cognitive load and make the
+# code less readable.
 @dataclass
 class Source:
     source: str

eventsourcingdb_client_python/handlers/observe_events/observe_events.py

@@ -18,7 +18,11 @@
 from .observe_events_options import ObserveEventsOptions
 from ...http_client.response import Response
 
-
+# pylint: disable=R6007
+# Reason: This method explicitly specifies the return type as None
+# for better readability. Even though it is not necessary,
+# it makes the return type clear without needing to read any
+# documentation or code.
 async def observe_events(
     client: AbstractBaseClient,
     subject: str,

eventsourcingdb_client_python/handlers/read_event_types/read_event_types.py

@@ -12,7 +12,11 @@
 from ...errors.validation_error import ValidationError
 from ...http_client.response import Response
 
-
+# pylint: disable=R6007
+# Reason: This method explicitly specifies the return type as None
+# for better readability. Even though it is not necessary,
+# it makes the return type clear without needing to read any
+# documentation or code.
 async def read_event_types(
     client: AbstractBaseClient,
 ) -> AsyncGenerator[EventType, None]:

eventsourcingdb_client_python/handlers/read_events/read_events.py

@@ -17,7 +17,11 @@
 from ..store_item import StoreItem
 from .read_events_options import ReadEventsOptions
 
-
+# pylint: disable=R6007
+# Reason: This method explicitly specifies the return type as None
+# for better readability. Even though it is not necessary,
+# it makes the return type clear without needing to read any
+# documentation or code.
 async def read_events(
     client: AbstractBaseClient,
     subject: str,

eventsourcingdb_client_python/handlers/read_subjects/read_subjects.py

@@ -14,7 +14,11 @@
 from .is_subject import is_subject
 from ...event.validate_subject import validate_subject
 
-
+# pylint: disable=R6007
+# Reason: This method explicitly specifies the return type as None
+# for better readability. Even though it is not necessary,
+# it makes the return type clear without needing to read any
+# documentation or code.
 async def read_subjects(
     client: AbstractBaseClient,
     base_subject: str