Skip to content

[TASK] Commits #5 - #614

Open
Shn4yder wants to merge 15 commits into
SENATOROVAI:mainfrom
Shn4yder:TASK-Commits-5
Open

[TASK] Commits #5#614
Shn4yder wants to merge 15 commits into
SENATOROVAI:mainfrom
Shn4yder:TASK-Commits-5

Conversation

@Shn4yder

Copy link
Copy Markdown
Member

@Shn4yder
Shn4yder requested a review from solik220-bit May 10, 2026 15:49

@Shn4yder Shn4yder left a comment

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.

сделайте ревью, пожалуйста

Comment thread python/commits.py
Comment on lines +1 to +39
"""Ответы на вопросы по TASK 5."""

# **1. Опишите своими словами назначение каждого из этих типов коммитов:**
# - feat - добавление новой функциональности (feature)
# - fix - решение какой-либо неисправности (бага)
# - docs - оформление/изменение документации
# - style - правки, которые не влияют на логику работы кода: исправление отступов, удаление лишних пробелов, приведение форматирования к единому стандарту и др.
# - refactor - рефакторинг кода - изменение структуры проекта, без изменения функционала
# - test - работа с тестами, при этом основной код не затрагивается
# - build - изменениям в системе сборки или внешних зависимостях (добавление/удаление пакетов)
# - ci - настройка непрерывной интеграции и доставки: изменения в конфигурации, сюда же — правки скриптов деплоя или проверок, которые запускаются автоматически
# - perf - изменения, которые улучшают производительность кода, скорость работы, потребление памяти, время загрузки
# - chore - коммит, который не устраняет баг и не вносит новый функционал, а модифицирует или обновляет зависимости
#

# **2. Представьте, что вы исправили баг в функции, которая некорректно округляет числа. Сделайте фиктивный коммит и напишите для него сообщение в соответствии с Conventional Commits (используя тип fix).**
#
# Сообщение для коммита: fix: correct rounding function for numbers

# **3. Добавление новой функциональности:
# Допустим, вы реализовали новую функцию generateReport в проекте. Сделайте фиктивный коммит с типом feat, отражающий добавление этой функциональности**
#
# Сообщение для коммита: feat: added generateReport function for exporting data

# **4. Модификация формата кода или стилей:
# Представьте, что вы поправили отступы и форматирование во всём проекте, не меняя логики кода. Сделайте фиктивный коммит с типом style**
#
# Сообщение для коммита: style: normalize indentation and formatting in project

# **5. Сделайте фиктивный коммит с типом docs, добавляющий или улучшающий документацию для вашей новой функции.**
#
# Сообщение для коммита: docs: add description for generateReport function

# **6. Сделайте фиктивный коммит с типом test, добавляющий тесты для этой же функции.**
#
# Сообщение для коммита: test: add tests for generateReport function

# **7. Отправьте сюда в комменты ссылку на ваши коммиты https://github.com/ВАШ-ЛОГИН/Data-Science-For-Beginners-from-scratch-SENATOROV/commits/main/**
# https://github.com/Shn4yder/Data-Science-For-Beginners-from-scratch-SENATOROV/commits/main

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.

сделайте ревью, пожалуйста

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.

добавление новой функции лучше писать так: feat: add generateReport function for exporting data.
а так всё верно!

@ruslansenatorov

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 11e202a0bc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread python/cpython.py
print("Количество файлов на C в папке include:", count)


count_h_files("C:\\Users\\user\\AppData\\Local\\Programs\\Python\\Python312\\include")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Avoid hard-coding a local CPython include path

When this script/notebook is run on any checkout that does not have exactly C:\Users\user\AppData\Local\Programs\Python\Python312\include, os.walk silently yields no files and the answer prints 0 instead of the recorded count of 215. Use a portable include path such as sysconfig.get_paths()["include"], or validate that the provided directory exists before counting, so rerunning the assignment reproduces the documented result.

Useful? React with 👍 / 👎.

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