Skip to content

style(docs): исправить format:check для visual comparison data - #404

Merged
TThanos3000 merged 1 commit into
devfrom
fix/desktop-comparison-format
Oct 5, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
fix/desktop-comparison-format

Conversation

@TThanos3000

Copy link
Copy Markdown
Contributor

Что изменено

Отформатирован docs/desktop-regression/comparison-data.js по конфигурации Prettier проекта. Исправляет падение npm run format:check после merge #403 в dev.

Зачем

CI сообщал о нарушении форматирования в единственном файле с результатами визуальных сравнений.

Реализация

Только форматирование одного файла. Проверка через выполнение исходного и отформатированного JS подтвердила идентичность данных: 36 desktop и 42 mobile/tablet сравнения.

API

Не менялся: diff затрагивает только данные отчёта в docs.

UI

Не менялся: код платформы и значения данных визуального отчёта не изменены.

Permissions

Не менялись: server-side код и правила доступа не затронуты.

Проверки

  • npm run format:check — exit 0, чистая копия всех tracked файлов с окончаниями строк из Git, как в CI.
  • Семантическое сравнение JS до/после — данные идентичны (36 desktop, 42 mobile/tablet).
  • git diff --check — без ошибок.

Acceptance criteria

  • Файл проходит Prettier; полный format check проходит.
  • Результаты визуальных сравнений и runtime код не изменены.

Не проверено

Runtime tests/build и повторная браузерная проверка неприменимы к форматированию данных отчёта. Новый запуск GitHub CI будет выполнен при открытии PR.

Риски / migration

Миграции не требуются. Область изменения ограничена форматированием одного файла; данные сохранены без изменений.

@TThanos3000
TThanos3000 merged commit 706a574 into dev Oct 5, 2026
@TThanos3000
TThanos3000 deleted the fix/desktop-comparison-format branch October 5, 2026 13:58
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.

2 participants