Configure flake8 to ignore unused imports in inits

[MorrisNein]

Added a configuration in setup.cfg for flake8 to ignore F401 in */__init__.py files

[pep8speaks]

Hello @MorrisNein! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file meta_automl/data_preparation/dataset/__init__.py:

Line 1:1: F401 '.dataset_base.DatasetBase' imported but unused
Line 1:1: F401 '.dataset_base.DatasetData' imported but unused
Line 1:1: F401 '.dataset_base.DatasetIDType' imported but unused
Line 1:1: F401 '.dataset_base.TabularData' imported but unused
Line 1:1: F401 '.dataset_base.TimeSeriesData' imported but unused
Line 2:1: F401 '.custom_dataset.CustomDataset' imported but unused
Line 2:1: F401 '.custom_dataset.DataNotFoundError' imported but unused
Line 3:1: F401 '.openml_dataset.OpenMLDataset' imported but unused
Line 3:1: F401 '.openml_dataset.OpenMLDatasetIDType' imported but unused
Line 4:1: F401 '.time_series_dataset.TimeSeriesDataset' imported but unused

Comment last updated at 2023-12-18 20:13:10 UTC

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7fb2990) 30.29% compared to head (614ede7) 30.29%.

❗ Current head 614ede7 differs from pull request most recent head 66a39da. Consider uploading reports for the commit 66a39da to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   30.29%   30.29%           
=======================================
  Files          56       56           
  Lines        2340     2340           
=======================================
  Hits          709      709           
  Misses       1631     1631           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MorrisNein]

@DRMPN, кажется, не получилось настроить pep8speaks. Хотя в README репозитория написано, что он подтягивает параметры из setup.cfg. Я провалидировал на локалке, flake8 воспринимает этот параметр. Может быть, ботом использованный параметр не поддерживается? Или есть какие-то идеи на этот счёт?

[DRMPN]

Или есть какие-то идеи на этот счёт?

В конфигурации указан порядок приоритезации конфигурации:

1. .pep8speaks.yml
2. setup.cfg

Так что он настройки второго скорее всего просто игнорит.
Можно попробовать разместить все настройки в 1 файле, если они не сработают, то переместить оттуда секцию flake8 во 2 файл.

[MorrisNein]

Мм. Я это ошибочно понял так, что он читает оба места, но применяет сначала одни настройки, а потом оставшиеся

Я пробовал задать этот параметр в .pep8speaks.yml, но это не возымело эффекта. Мб я проверял неправильно.

Судя по дефолтному конфигу из того же репо, он вообще там не предусмотрен

[MorrisNein]

...переместить оттуда секцию flake8 во 2 файл.

Не сработало

[DRMPN]

Бегло посмотрел код бота, он должен передавать настройки из .pep8speaks.yml и setup.cfg в аргументы к линтеру.

Не сработало

Попробуй проверить, подхватил ли он ограничение в 120 символов. Может путь указан некорректно?