Skip to content

Conversation

@osogi
Copy link
Owner

@osogi osogi commented Sep 7, 2024

The first task is completed:

  • two functions specified in the task are implemented
  • unit tests are written for them

@osogi osogi requested a review from WoWaster September 8, 2024 20:04
Copy link
Collaborator

@WoWaster WoWaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

С задачей всё хорошо. Пару слов про линтер и форматтер: тут в CI используется ruff, который идёт в комплекте с rye. Поэтому лучше прикрутить его. Если потестишь конфиг от Георгия Белянина, будет классно FormalLanguageConstrainedPathQuerying#170.
Изменения можно либо в эту ветку, либо сразу в main

@osogi
Copy link
Owner Author

osogi commented Sep 9, 2024

...тут в CI используется ruff, который идёт в комплекте с rye. Поэтому лучше прикрутить его

Я наверное совмещу flake8 и rye, так как где-то в ридми сказано, что придерживаемся pep8. А у меня с его неймингом бывают проблемы. Подгружу скорее всего в эту ветку

@WoWaster
Copy link
Collaborator

WoWaster commented Sep 9, 2024

ruff вроде как вбирает в себя и правила flake8. Но дело твоё. Главное, чтобы на CI была галочка :)

@osogi
Copy link
Owner Author

osogi commented Sep 9, 2024

Проверю, включает ли ruff pep8-naming

@osogi osogi merged commit d3589b2 into main Sep 13, 2024
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.

3 participants