Skip to content

[TASK] Commits #5 (https://github.com/SENATOROVAI/intro-cs/issues/5)#515

Closed
Otabek1121 wants to merge 12 commits intoSENATOROVAI:mainfrom
Otabek1121:main
Closed

[TASK] Commits #5 (https://github.com/SENATOROVAI/intro-cs/issues/5)#515
Otabek1121 wants to merge 12 commits intoSENATOROVAI:mainfrom
Otabek1121:main

Conversation

@Otabek1121
Copy link
Copy Markdown

@Otabek1121 Otabek1121 commented Jan 7, 2026

Describe your change:

[TASK] Commits #5

Fixes #5

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Add or change doctests? -- Note: Please avoid changing both code and tests in a single pull request.
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new Python files are placed inside an existing directory.
  • All filenames are in all lowercase characters with no spaces or dashes.
  • All functions and variable names follow Python naming conventions.
  • All function parameters and return values are annotated with Python type hints.
  • All functions have doctests that pass the automated testing.
  • All new algorithms include at least one URL that points to Wikipedia or another similar explanation.
  • If this pull request resolves one or more open issues then the description above includes the issue number(s) with a closing keyword: "Fixes #ISSUE-NUMBER".

Copy link
Copy Markdown
Author

@Otabek1121 Otabek1121 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 commits.py Outdated
Comment on lines +1 to +50
# %% [markdown]
# """Task commits."""

# %% [markdown]
# 1. Опишите своими словами назначение каждого
# из этих типов коммитов:

# %% [markdown]
# feat - добавляет новую фичу в код
#
# fix - исправляет баг в коде
#
# docs - такой нужен в случае,
# если изменения только в
# документации
#
# style - не влияет на сам код
#
# refactor - помогает сделать код чище, т.е не влияет на фичи и
# не чинит баг. Но делает код более читабельным и понятным.
#
# test - добавляет тесты или исправляет существующие
#
# build - влияет на систему сборки или внешние зависимости
#
# ci - добавляет изменения в файлах конфигурации CI и скриптах.
#
# То есть настраивает изменение процессов
# сборки, тестирования и деплоя.
#
# perf - улучшается производительность
#
# chore - нужен для технического обслуживания проекта
#

# %% [markdown]
# 2) git commit -m "fix: rounding error in round function
# Now it correctly rounds numbers"
#
#
#

# %% [markdown]
# 3. git commit -m "feat: add generateReport function"

# %% [markdown]
# 4. git commit -m "style: fix indentation and formatting"

# %% [markdown]
# 5. git commit -m "docs: update documentation for generateReport function"
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

please do a review

Comment thread commits.py
@@ -0,0 +1,50 @@
# %% [markdown]
# """Task commits."""
Copy link
Copy Markdown
Member

@stokato stokato Jan 7, 2026

Choose a reason for hiding this comment

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

Ячейка должна быть типом python
Иначе линтер будет ругаться nbqa-pydocstyle D100

Copy link
Copy Markdown
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
Author

Choose a reason for hiding this comment

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

Как в ячейку python напишу текст? Ошибку будет выдавать

Comment thread commits.py Outdated
# из этих типов коммитов:

# %% [markdown]
# feat - добавляет новую фичу в код
Copy link
Copy Markdown
Member

@stokato stokato Jan 7, 2026

Choose a reason for hiding this comment

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

фичу в приложении

Comment thread commits.py Outdated
# %% [markdown]
# feat - добавляет новую фичу в код
#
# fix - исправляет баг в коде
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.

баг в приложении

@stokato
Copy link
Copy Markdown
Member

stokato commented Jan 7, 2026

Формат в инструкции для pr

Заголовок коммита:
Имя ишюьса (ссылка на ишьюс в круглых скабках)

Описание коммита:
Closes ссылка на ишьюс

Описание в pr, такое и должно быть?

@stokato
Copy link
Copy Markdown
Member

stokato commented Jan 7, 2026

Так же я не вижу комментария вот в этом ишью 5
с твоими фиктивными коммитами
https://github.com/Otabek1121/Data-Science-For-Beginners-from-scratch-SENATOROV/commits/main/

Comment thread commits.py Outdated
# 4. git commit -m "style: fix indentation and formatting"

# %% [markdown]
# 5. git commit -m "docs: update documentation for generateReport function"
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.

не хватает команды для создания фиктивного коммита под test

Comment thread commits.py Outdated
# если изменения только в
# документации
#
# style - не влияет на сам код
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.

Поправь пожалуйста описание, не совсем понятно, на что-же все таки оказывает влияние

@stokato
Copy link
Copy Markdown
Member

stokato commented Jan 8, 2026

Еще нюанс, сделай чтобы линтеры у тебя не падали в ошибку - это важно!

@Otabek1121
Copy link
Copy Markdown
Author

занова сделаю

@Otabek1121 Otabek1121 closed this Jan 8, 2026
Copy link
Copy Markdown
Author

@Otabek1121 Otabek1121 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 commits.py
Comment on lines +1 to +43
# %% [markdown]
# """Task commits."""

# %% [markdown]
# 1. Опишите своими словами назначение каждого
# из этих типов коммитов:

# %% [markdown]
#
# - feat,Добавление новой возможности или функции для пользователя.
# - fix,Работа над ошибками: исправление багов и поломок в коде.
# - docs,"Всё, что касается текста: документация, README, комментарии в коде."
# - style,"«Косметический ремонт»: правка отступов, кавычек и форматирования без изменения логики."
# - refactor,"Переписывание кода для красоты и порядка, не меняя его поведения (не фича и не фикс)."
# - test,Работа с проверками: добавление новых тестов или исправление существующих.
# - build,"Изменения, влияющие на сборку проекта или внешние зависимости (npm, pip, и т.д.)."
# - ci,"Настройка «автопилота»: изменения в скриптах автоматизации (GitHub Actions, GitLab CI)."
# - perf,"Работа над скоростью: изменения, которые заставляют код работать быстрее или потреблять меньше памяти."
# - chore (рутинная работа),"Прочие задачи: обновление .gitignore, мелкие правки, не относящиеся к самому коду."

# %% [markdown]
# 2.Как правильно записать коммит об исправлении бага с округлением?
# fix: correct rounding logic in the calculation function

# %% [markdown]
# 3. Добавление новой функции (feat)
#
# feat: implement generateReport function for automated PDF exports

# %% [markdown]
# 4. Модификация стиля кода (style)
# Исправлены отступы во всем проекте (например, переход с 2 пробелов на 4).
#
# style: fix indentation and reformat source code using PEP8 standards

# %% [markdown]
# 5.Документация и обучение:
#
# Сделайте воображаемый коммит с типом документов, добавляя или улучшая документацию для вашей новой функции.
# Сделайте вымышленный коммит с типом теста, добавив тесты для этой же функции.
#
# docs: add documentation and usage examples for generateReport function
# test: add unit tests to verify generateReport output accuracy
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

на ревю

@Otabek1121 Otabek1121 reopened this Jan 8, 2026
@stokato
Copy link
Copy Markdown
Member

stokato commented Jan 8, 2026

У тебя падает линтер
nbqa-pydocstyle..........................................................Failed

  • hook id: nbqa-pydocstyle
  • exit code: 1

docs/git/git_command.ipynb:cell_1:1 at module level:
D213: Multi-line docstring summary should start at the second line
lesson1.ipynb:cell_1:1 at module level:
D205: 1 blank line required between summary line and description (found 0)
lesson1.ipynb:cell_1:1 at module level:
D209: Multi-line docstring closing quotes should be on a separate line
lesson1.ipynb:cell_1:1 at module level:
D213: Multi-line docstring summary should start at the second line
log.ipynb:cell_1:1 at module level:
D205: 1 blank line required between summary line and description (found 0)
log.ipynb:cell_1:1 at module level:
D209: Multi-line docstring closing quotes should be on a separate line
log.ipynb:cell_1:1 at module level:
D213: Multi-line docstring summary should start at the second line

https://github.com/SENATOROVAI/Data-Science-For-Beginners-from-scratch-SENATOROV/actions/runs/20818385888/job/59799924039?pr=515

@stokato
Copy link
Copy Markdown
Member

stokato commented Jan 8, 2026

Так же я не вижу комментария вот в этом ишью 5 с твоими фиктивными коммитами https://github.com/Otabek1121/Data-Science-For-Beginners-from-scratch-SENATOROV/commits/main/

вот это не выполнено

@Otabek1121 Otabek1121 closed this Jan 9, 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.

2 participants