Skip to content

Fix/issue4 [TASK] Cpython (https://github.com/SENATOROVAI/intro-cs/issues/4)#522

Closed
dshilin wants to merge 4 commits intoSENATOROVAI:mainfrom
dshilin:fix/issue4
Closed

Fix/issue4 [TASK] Cpython (https://github.com/SENATOROVAI/intro-cs/issues/4)#522
dshilin wants to merge 4 commits intoSENATOROVAI:mainfrom
dshilin:fix/issue4

Conversation

@dshilin
Copy link
Copy Markdown
Member

@dshilin dshilin commented Jan 22, 2026

@dshilin dshilin changed the title Fix/issue4 [TASK] Cpython (#4) (https://github.com/SENATOROVAI/intro-cs/issues/4) Fix/issue4 [TASK] Cpython #4 (https://github.com/SENATOROVAI/intro-cs/issues/4) Jan 22, 2026
@dshilin dshilin changed the title Fix/issue4 [TASK] Cpython #4 (https://github.com/SENATOROVAI/intro-cs/issues/4) Fix/issue4 [TASK] Cpython (https://github.com/SENATOROVAI/intro-cs/issues/4) Jan 22, 2026
@stokato
Copy link
Copy Markdown
Member

stokato commented Jan 22, 2026

Closes (SENATOROVAI/intro-cs#4) Fixed #4

SENATOROVAI/intro#5
Оформление не совсем совпадает

по инструкции где ты выделяешь код, который должен заревьювить ревьювер?
Посмотри на 3 пункт внимательнее

Comment thread python/cpython.py
#
# Задание 2: Исследование структуры CPython
#
# Прочитал README проекта
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Какое количество файлов на C там есть?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

серьезно? там есть 62 файла

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Получается серьезно, задача есть и игнорить её не лучшее решение.

Comment thread python/cpython.py
#
# Прочитал README проекта
#
# Задание 3: Запуск файла с помощью CPython
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image Задача и решение не совпадают, сделай выводы хотя-бы так, что консолька выдает.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

image

Copy link
Copy Markdown
Member Author

@dshilin dshilin left a comment

Choose a reason for hiding this comment

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

прокомментировал комментарии

Comment thread python/cpython.py
#
# Прочитал README проекта
#
# Задание 3: Запуск файла с помощью CPython
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

image

Comment thread python/cpython.py
#
# Задание 2: Исследование структуры CPython
#
# Прочитал README проекта
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

серьезно? там есть 62 файла

Copy link
Copy Markdown
Member

@stokato stokato left a comment

Choose a reason for hiding this comment

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

review done

@stokato stokato closed this Jan 23, 2026
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.

3 participants