Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Slugify URLs and make orgs & repos info updatable #109

Merged
merged 4 commits into from Oct 15, 2021
Merged

Conversation

antlu
Copy link
Collaborator

@antlu antlu commented Jan 18, 2021

Заменил ID в URL-ах ...organizations/2685047 на имена ...organizations/Hexlet.
Сделал данные организаций и репозиториев обновляемыми на случай смены имени, например.

@astynax
Copy link
Contributor

astynax commented Feb 2, 2021

Вижу в нескольких местах изменения, заключающиеся только в изменении noqa. Чем эти изменения вызваны? Сборка без них теперь падает из-за обновления линтера?

@antlu
Copy link
Collaborator Author

antlu commented Feb 2, 2021

Здесь проверка пройдёт. Это у меня локально линтер обновлён, видимо, и выдаёт новые предупреждения, которые я и включил в noqa.

@astynax
Copy link
Contributor

astynax commented Feb 2, 2021

Нужно разрабатывать с тем же линтером, что и в проекте используется. Не должно быть в коде чего-то, завязанного на среду разработки кого-то из контрибуторов.

@antlu
Copy link
Collaborator Author

antlu commented Feb 2, 2021

Да я уже после пуша обнаружил эту разницу, исправлю.

@antlu antlu marked this pull request as draft February 2, 2021 06:38
@antlu antlu marked this pull request as ready for review February 2, 2021 08:13
@antlu
Copy link
Collaborator Author

antlu commented Feb 2, 2021

Перепроверил версию линтера – у меня 0.14, как и здесь, но выдаёт предупреждения, а на Гитхабе их нет.

./contributors/utils/github_lib.py
  8:1      S105  Possible hardcoded password: 'https://github.com/login/oauth/access_token'

./contributors/management/commands/fetchdata.py
  36:1     R701  'create_contributions' is too complex (12)
  92:5     A003  "help" is a python builtin, consider renaming the class attribute
  136:29   G004  Logging statement uses f-string

@astynax
Copy link
Contributor

astynax commented Feb 2, 2021

Сдаётся мне, что линтер не работает. А из-за @ в Makefile не видно, где он запускается и запускается ли.

@antlu
Copy link
Collaborator Author

antlu commented Feb 2, 2021

Он запускается, выдаёт 0 замечаний в 11 строке лога.

@astynax
Copy link
Contributor

astynax commented Feb 2, 2021

Хмм. Значит либо у вас не подхватывает какие-то исключения из setup.cfg, либо ещё где-то разница есть. Вы точно запускаете у себя линтер командой make lint?

@antlu
Copy link
Collaborator Author

antlu commented Feb 2, 2021

Да, make lint. Оказывается дело в версии Питона. Сделал копию проекта, и линтер показал только S105 на 3.6.9, на 3.9.1 показывает WPS221, а на 3.8.7 (как тут) замечаний нет.

@astynax
Copy link
Contributor

astynax commented Feb 2, 2021

Стоит погонять actions на разных пайтонах, чтоли...
@zhabinka как думаешь, можно в action дописать такое?

@antlu antlu closed this Feb 4, 2021
@antlu antlu deleted the slug branch February 4, 2021 06:21
@antlu antlu restored the slug branch February 4, 2021 06:39
@antlu antlu reopened this Feb 4, 2021
@astynax astynax merged commit 3e372b6 into Hexlet:master Oct 15, 2021
@antlu antlu mentioned this pull request Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants