argilla
argilla copied to clipboard
[REFACTOR] Align `@abstractmethod` definition within `argilla`
Is your feature request related to a problem? Please describe.
As of the Python documentation the usage of @abstractproperty
is discouraged and deprecated since Python 3.3, so we should stop using those in the first place, replacing them with @property
+ @abstractmethod
.
Then regarding the body of those @abstractmethod
s we should define an standard way of defining those.
Describe the solution you'd like
-
Adding a one-line docstring
from abc import abstractmethod class A: ... @abstractmethod def implement_me(self) -> None: """This is a docstring"""
-
Adding
pass
from abc import abstractmethod class A: ... @abstractmethod def implement_me(self) -> None: pass
-
Adding
...
from abc import abstractmethod class A: ... @abstractmethod def implement_me(self) -> None: ...
@frascuchon @jfcalvo @gabrielmbmb
This issue is stale because it has been open for 90 days with no activity.
This issue was closed because it has been inactive for 30 days since being marked as stale.