Skip to content

544_table_percentage#760

Merged
HadronCollider merged 7 commits intodevfrom
544_table_percentage
Mar 16, 2026
Merged

544_table_percentage#760
HadronCollider merged 7 commits intodevfrom
544_table_percentage

Conversation

@vovanbravin
Copy link
Collaborator

@vovanbravin vovanbravin commented Nov 21, 2025

@vovanbravin vovanbravin linked an issue Dec 7, 2025 that may be closed by this pull request
@HadronCollider HadronCollider changed the base branch from master to dev December 7, 2025 10:39
Copy link
Collaborator

@HadronCollider HadronCollider left a comment

Choose a reason for hiding this comment

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

  • подтяните изменения из dev и проверьте работу критерия (у меня он почему-то не запустился)

Comment on lines +13 to +16
class TablePercentage(BaseReportCriterion):
label = "Проверка процентного соотношения таблиц в документе"
description = "Проверяет, что таблицы занимают не более установленного процента площади документа"
id = 'table_percentage'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Придерживайтесь шаблонов именования

  • классы - Pres<_>Check
  • id = pres_<>_check

Copy link
Collaborator

Choose a reason for hiding this comment

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

Отформатируйте код по pep8

@vovanbravin
Copy link
Collaborator Author

@HadronCollider Здравствуйте, я проверил на dev, у меня критерий запускается и работает. Может вы его не заметили, у меня он шел 15.

Copy link
Collaborator

@HadronCollider HadronCollider left a comment

Choose a reason for hiding this comment

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

Нет проверки на большие таблицы (занимающие больше N страниц)

@github-actions github-actions bot added the has conflicts if new merge has conflicts label Mar 16, 2026
@github-actions github-actions bot removed the has conflicts if new merge has conflicts label Mar 16, 2026
@HadronCollider HadronCollider merged commit c1538d0 into dev Mar 16, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Критерий проверки доли таблиц

2 participants