ci: тихий Format check и честная кодировка (XIC-52) - #68
Merged
Merged
Conversation
Шаг Format check (non-blocking) печатал ~205 замечаний и горел красным, хотя реального дрейфа стиля не было: - CHARSET (~120): .editorconfig требовал utf-8-bom, а 120 из 149 .cs сохранены без BOM. Правило приведено к факту (utf-8), BOM снят с оставшихся 29 файлов — единообразно. - RCS1139 (7): doc-комментарий из одного <param> — стиль проекта, правило заглушено в .editorconfig с обоснованием. - WHITESPACE: шесть файлов с намеренным ручным выравниванием исключены через --exclude; TouchpadHaptics.cs просто отформатирован. Список исключений синхронизирован с CLAUDE.md/CONTRIBUTING. Там же — заметка, что красный Snyk на PR из форка ожидаем (не может склонировать форк) и мержу не мешает. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ф стиля (XIC-52) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Закрывает XIC-52.
Шаг Format check (non-blocking) печатал ~205
##[error]и горел красным при нулевом реальном дрейфе стиля. Разбор по видам замечаний:.editorconfigтребовалutf-8-bom, а 120 из 149.csбез BOMutf-8, BOM снят с 29 файлов<param>— стиль проекта.editorconfigс обоснованием--excludeвci.yml;TouchpadHaptics.csотформатированСписок исключений синхронизирован с
CLAUDE.mdиCONTRIBUTING.md. В CONTRIBUTING также записано, что красныйsecurity/snykна PR из форка — сбой клонирования, а не уязвимость.Проверка: локальный
dotnet format --verify-no-changesс исключениями — только ENDOFLINE от LF в рабочей копии (в CI checkout даёт CRLF); 719 тестов зелёные. Итог по шагу — смотреть в CI этого PR.🤖 Generated with Claude Code