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

Black, flake8 were run for vkwave #146

Merged
merged 1 commit into from
Aug 29, 2021
Merged

Black, flake8 were run for vkwave #146

merged 1 commit into from
Aug 29, 2021

Conversation

doubledare704
Copy link

@doubledare704 doubledare704 commented Aug 26, 2021

Типы изменений

  • Багфикс (обратно-совместимое изменение, исправляющее ишью)
  • Новый функционал (обратно-совместимое изменение, добавляющее что-то новое)
  • Кардинальное изменение (фикс или фишка, ломающая обратную совместимость)
  • Я прочел CONTRIBUTING.
  • Я прочел CODE_OF_CONDUCT.
  • Мой код соответствует принятому в проекте стилю кода.
  • Мои изменения требуют изменений в документации.
  • Я обновил документацию.
  • Я добавил тесты на мои изменения.
  • Все новые и уже существующие тесты проходят.

I've updated formatting and made small changes according to flake8 errors.
PR is related to issue #145

@dadyarri
Copy link
Collaborator

Добрый день, спасибо за вклад в развитие библиотеки!

В flake8 исправлены не все ошибки:
image
Не все файлы форматированы по black:
image

(Для решения достаточно перезапустить black)

@dadyarri dadyarri merged commit f2a9ea6 into prostomarkeloff:master Aug 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants