Remove pylint from the project

tests/console/test_main.py

@@ -18,8 +18,6 @@ class TestShowVersion:
     " (development)" to the output.
     """
 
-    # pylint:disable=no-self-use
-
     @pytest.fixture
     def ctx(self) -> click.Context:
         """Context around the `main.entry_point` Command."""
@@ -65,8 +63,6 @@ class TestCLIWithoutCommand:
     Therefore, they are not considered for coverage reporting.
     """
 
-    # pylint:disable=no-self-use
-
     @pytest.mark.no_cover
     def test_no_options(self, cli):
         """Exit with 0 status code and no output if run without options."""

tests/db/fake_data/factories.py

@@ -121,8 +121,6 @@ class AdHocOrderFactory(alchemy.SQLAlchemyModelFactory):
     flags to adapt how the `Order` is created.
     """
 
-    # pylint:disable=too-many-instance-attributes
-
     class Meta:
         model = db.Order
         sqlalchemy_get_or_create = ('id',)
@@ -282,7 +280,7 @@ class AdHocOrderFactory(alchemy.SQLAlchemyModelFactory):
     )
 
     @factory.post_generation
-    def post(  # noqa:C901,WPS23 pylint:disable=unused-argument
+    def post(  # noqa:C901,WPS231
         obj, create, extracted, **kwargs,  # noqa:B902,N805
     ):
         """Discard timestamps that occur after cancellation."""

tests/db/test_addresses.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Address` model."""
-# pylint:disable=no-self-use,protected-access
 
 import pytest
 import sqlalchemy as sqla

tests/db/test_addresses_pixels.py

@@ -8,7 +8,6 @@ Implementation notes:
     code associated with it. Therefore, we test it here as non-e2e tests and do
     not measure its coverage.
 """
-# pylint:disable=no-self-use
 
 import pytest
 import sqlalchemy as sqla

tests/db/test_cities.py

@@ -1,5 +1,4 @@
 """Test the ORM's `City` model."""
-# pylint:disable=no-self-use
 
 import pytest
 

tests/db/test_couriers.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Courier` model."""
-# pylint:disable=no-self-use
 
 import pytest
 from sqlalchemy import exc as sa_exc

tests/db/test_customer.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Customer` model."""
-# pylint:disable=no-self-use
 
 import pytest
 

tests/db/test_forecasts.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Forecast` model."""
-# pylint:disable=no-self-use
 
 import datetime
 

tests/db/test_grids.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Grid` model."""
-# pylint:disable=no-self-use
 
 import pytest
 import sqlalchemy as sqla

tests/db/test_orders.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Order` model."""
-# pylint:disable=no-self-use,protected-access
 
 import datetime
 import random
@@ -49,8 +48,6 @@ class TestProperties:
     and provided by the `make_order` fixture.
     """
 
-    # pylint:disable=no-self-use,too-many-public-methods
-
     def test_is_ad_hoc(self, order):
         """Test `Order.scheduled` property."""
         assert order.ad_hoc is True

tests/db/test_pixels.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Pixel` model."""
-# pylint:disable=no-self-use
 
 import pytest
 import sqlalchemy as sqla

tests/db/test_restaurants.py

@@ -1,5 +1,4 @@
 """Test the ORM's `Restaurant` model."""
-# pylint:disable=no-self-use
 
 import pytest
 import sqlalchemy as sqla

tests/db/utils/test_locations.py

@@ -1,5 +1,4 @@
 """Test the `Location` class."""
-# pylint:disable=no-self-use
 
 import pytest
 
@@ -97,7 +96,6 @@ class TestSpecialMethods:
 
     def test_compare_utm_coordinates_to_themselves(self, location):
         """Test `Location.__eq__()`."""
-        # pylint:disable=comparison-with-itself
         result = location == location  # noqa:WPS312
 
         assert result is True

tests/forecasts/timify/test_aggregate_orders.py

@@ -1,5 +1,4 @@
 """Test the `OrderHistory.aggregate_orders()` method."""
-# pylint:disable=no-self-use,unused-argument
 
 import datetime
 

tests/forecasts/timify/test_make_time_series.py

@@ -3,7 +3,6 @@
 Unless otherwise noted, each `time_step` is 60 minutes long implying
 12 time steps per day (i.e., we use `LONG_TIME_STEP` by default).
 """
-# pylint:disable=no-self-use,unused-argument
 
 import datetime
 
@@ -54,7 +53,7 @@ def order_totals(good_pixel_id):
 def order_history(order_totals, grid):
     """An `OrderHistory` object that does not need the database."""
     oh = timify.OrderHistory(grid=grid, time_step=test_config.LONG_TIME_STEP)
-    oh._data = order_totals  # pylint:disable=protected-access
+    oh._data = order_totals
 
     return oh
 

tests/forecasts/timify/test_order_history.py

@@ -1,5 +1,4 @@
 """Test the basic functionalities in the `OrderHistory` class."""
-# pylint:disable=no-self-use
 
 import pytest
 

tests/test_version.py

@@ -20,8 +20,6 @@ import urban_meal_delivery
 class TestPEP404Compliance:
     """Packaged version identifier is PEP440 compliant."""
 
-    # pylint:disable=no-self-use
-
     @pytest.fixture
     def parsed_version(self) -> str:
         """The packaged version."""
@@ -47,8 +45,6 @@ class TestPEP404Compliance:
 class TestSemanticVersioning:
     """Packaged version follows a strict subset of semantic versioning."""
 
-    # pylint:disable=no-self-use
-
     version_pattern = re.compile(
         r'^(0|([1-9]\d*))\.(0|([1-9]\d*))\.(0|([1-9]\d*))(\.dev(0|([1-9]\d*)))?$',
     )