Skip to content

768 content objects 3 level - #872

Open
LapshinAE0 wants to merge 12 commits into
devfrom
768_content_objects_3level
Open

LapshinAE0 wants to merge 12 commits into
devfrom
768_content_objects_3level

Conversation

@LapshinAE0

Copy link
Copy Markdown
Collaborator

No description provided.

@HadronCollider HadronCollider left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • параметр end_marker не используется
  • заблокируйте обработку для MD-документов (найдите метод is_markdown в file), т.к. для них не применимо
  • определите начало и конец раздела точнее - проверкой положительного forbidden_depth, включая значение 1 (текущий regex требует хотя бы две числовые компоненты)
  • исправьте {self.forbidden_depth - 1} в сообщении и показывать найденные проблемные строки
  • задокументируйте настройки критерия и оценку.

Тесты

  • основные параметризованные проверки глубины не собираются pytest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

2 participants