Skip to content

docs: documented utils.py#720

Merged
greenbonebot merged 3 commits into
greenbone:mainfrom
HoJLter:main
Apr 30, 2025
Merged

docs: documented utils.py#720
greenbonebot merged 3 commits into
greenbone:mainfrom
HoJLter:main

Conversation

@HoJLter

@HoJLter HoJLter commented Apr 29, 2025

Copy link
Copy Markdown
Contributor

added docstrings to utils.py module

*added docstrings to utils.py
@HoJLter HoJLter requested a review from a team as a code owner April 29, 2025 19:12
@greenbonebot greenbonebot enabled auto-merge (rebase) April 29, 2025 19:12
@github-actions

github-actions Bot commented Apr 29, 2025

Copy link
Copy Markdown

Conventional Commits Report

Type Number
Added 1

🚀 Conventional commits found.

@HoJLter HoJLter changed the title docs: documented the utils.py docs: documented utils.py Apr 29, 2025

@y0urself y0urself left a comment

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.

Hey @HoJLter thank you for your contribution.

You may want to add Args: and Returns: on all functions? :)
Thanks!

Comment thread autohooks/utils.py Outdated
auto-merge was automatically disabled April 29, 2025 20:08

Head branch was pushed to by a user without write access

@greenbonebot greenbonebot enabled auto-merge (rebase) April 29, 2025 20:08
@HoJLter

HoJLter commented Apr 29, 2025

Copy link
Copy Markdown
Contributor Author

Hey @HoJLter thank you for your contribution.

You may want to add Args: and Returns: on all functions? :) Thanks!

:)

y0urself
y0urself previously approved these changes Apr 29, 2025

@y0urself y0urself left a comment

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.

Noice! Thank you very much!

Edit: Oh, guess black is not as happy, as I am. :/

auto-merge was automatically disabled April 29, 2025 21:02

Head branch was pushed to by a user without write access

@greenbonebot greenbonebot enabled auto-merge (rebase) April 29, 2025 21:02
@HoJLter

HoJLter commented Apr 29, 2025

Copy link
Copy Markdown
Contributor Author

i'll make him happy

@greenbonebot greenbonebot merged commit 8eb9fea into greenbone:main Apr 30, 2025
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