Repository navigation
FEEDBACK-443: вернуть неаккуратные примеры в урок и задание про линтер - #399
Merged
Merged
Conversation
Прогон форматтера привёл к стилю оба блока урока, и он показывал одинаковый код там, где текст обещает разницу. Заодно из примера пропали табуляция и закрывающий тег `?>`, хотя абзац ниже перечисляет ровно эти нарушения как находки линтера. В задании вышло хуже: условие просит исправить пробелы вокруг операторов, а код под ним прогон уже исправил. Выполнить такое задание нельзя. Неаккуратные версии возвращены во всех трёх локалях, блоки помечены как антипримеры (`docs/markdown.md` кита, § «Антипример идёт в ```text»). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
… арифметику Абзац говорит «вариант без пробелов тоже работает», а под ним лежал код с пробелами: прогон форматтера привёл к стилю оба блока. Так во всех трёх локалях. Найдено детектором пар `bin/check-twin-blocks.ts` в ките: обход по именам уроков этот урок не ловил, он не про линтер. Блоки помечены как антипримеры. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Абзац говорит «можно использовать одинарные или двойные», а оба примера под ним стояли в одинарных: прогон форматтера свёл обе формы к одной. Показывать было нечего. Рекомендация урока при этом верна и остаётся как есть: mago, который ведёт php в ките, сводит строки к одинарным кавычкам — проверено прогоном. Правится только демонстрация второй формы. Co-Authored-By: Claude Opus 5 (1M context) <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
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.
Парная правка к PR в
exercises-javascript(#867) и к правкам курсов Хекслетаphp_fundamentalsвruиes. Тикет FEEDBACK-443.Что случилось
Прогон
make lint-code-blocks-fix(кампанияmdsf-code-blocksв ките) привёл к стилю оба блока урока.README — два места:
Урок показывает этот блок под «Рассмотрим пример», а ниже объясняет, что линтер укажет на нехватку пробелов. Указывать стало не на что.
Второй блок пострадал сильнее. Абзац под ним перечисляет три находки линтера — табуляция вместо пробелов, запрещённый закрывающий тег
?>, пробел перед скобкой вызова. Прогон убрал из кода все три:Комментарий «На следующей строке отступ равен одному табу» остался и указывает на строку без табуляции.
EXERCISE — здесь задание стало невыполнимым. Условие просит «исправьте пробелы вокруг операторов», а код под ним уже исправлен:
Снаружи ничего не видно: markdown валиден, линтеры молчат,
lint-code-blocksзелёный по построению — форматтер сам это и сделал.Что в правке
Неаккуратные версии возвращены во всех трёх локалях, блоки переведены на метку
textи помечены:Правило закреплено в ките (
docs/markdown.md, § «Антипример идёт в ```text»), список дыр в проверке собирается грепом по этой строке. Цена — блок теряет подсветку; размен сознательный, разобран вdocs/mdsf-code-blocks.md.Проза не менялась: правка целиком внутри блоков кода.
🤖 Generated with Claude Code