fix(strings): FEEDBACK-424 вернуть формы кавычек, которые переписал форматтер - #381
Merged
Conversation
…орматтер Прогон mdsf свёл кавычки в блоках к двойным, а текст рядом продолжал описывать прежний код: список примеров стоял на одинарных, «можно и так, и так» показывало одну форму дважды, экранирование двойной кавычки стало экранированием апострофа, а «вариант без пробелов» шёл с расставленными пробелами. Канон курса — двойные кавычки: конфига ruff в репозитории нет, у решений практик 33 файла из 37 на двойных. Поэтому проза и схемы сведены к коду, а блоки, где форма записи и есть предмет урока, помечены text с NOTE: без пометки следующий прогон впишет версию форматтера обратно. Заодно убрано ложное «PEP 8 рекомендует одинарные кавычки»: PEP 8 не рекомендует ни те, ни другие и советует выбрать один стиль. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…и блоков Уроки показывали SyntaxError текстом Python 3.9 (EOL while scanning string literal), лишний заголовок Traceback и каретку не на том символе, хотя requires-python в репозитории >=3.11. Вывод снят прогоном python3. Транскрипт сессии с командой и выводом это console с промптом, а чистый вывод интерпретатора это text: под bash оба блока валили lint-code-blocks в «не проверено», потому что shfmt на них не работает. 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.
Тикет: FEEDBACK-424. Дефект нашёл студент: «у вас в этом курсе много раз напутано про двойные и одинарные кавычки».
Причина
Прогон
mdsf(ruff:format) переписал кавычки в блоках уроков на двойные, а текст рядом продолжал описывать прежний код. Класс задокументирован в ките (docs/mdsf-code-blocks.md, § «Форматтер переписывает то, что урок объясняет»): урок про кавычки там назван, но разобран не был.Канон курса — двойные кавычки: конфига ruff в репозитории нет, у решений практик 33 файла из 37 на двойных. Поэтому проза сведена к коду, а не наоборот.
Что было не так
25-strings/10-quotesprint("Hello")под фразой «можно и так, и так»; экранирование двойной кавычки переписано в экранирование апострофа, и абзац про\"повис; ложное «принято использовать одинарные, PEP8 придерживается этого стиля»10-basics/10-hello-world20-arithmetics/20-basic25-strings/20-string-concatenation25-strings/15-escape-characters30-variables/*,31-advanced-strings/70-slices31-advanced-strings/90-multiline-strings10-basics/50-syntax-errors,50-loops/90-debugEOL while scanning string literal), лишний заголовокTraceback, каретка не на том символе; транскрипт сессии под меткойbashКак починено
Блоки, где форма записи и есть предмет урока, помечены
```textс комментарием<!-- NOTE: … -->перед блоком. Без пометки правка не держится: следующий прогон форматтера впишет свою версию обратно.Вывод интерпретатора снят прогоном
python3(в репозиторииrequires-python >=3.11). Все восстановленные фрагменты прогнаны и дают то, что написано в комментариях# =>.Проверки
mdsf verify(черезmake lint-code-blocksкита) — зелёный по всем тронутым файлам;rumdl—No issues found in 413 files;python3.Парная правка на стороне Хекслета: MR в
courses/ru/python_fundamentals_courseиcourses/es/python_fundamentals_course.🤖 Generated with Claude Code