Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change: create_task optional argument name #1120

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ITSEC-DACHSER
Copy link

Added an optional argument to the function create_task to adjust the name

What

  • Optional parameter added to function create_task in scan-new-target.py

Why

  • Non-static names needed to fit in with naming convention
  • Change asked by Greenbone-Support

References

  • GS-6452

@ITSEC-DACHSER ITSEC-DACHSER requested a review from a team as a code owner July 9, 2024 09:20
@greenbonebot greenbonebot enabled auto-merge (rebase) July 9, 2024 09:20
Copy link

github-actions bot commented Jul 9, 2024

Conventional Commits Report

Type Number
Changed 1

🚀 Conventional commits found.

y0urself
y0urself previously approved these changes Jul 9, 2024
@y0urself
Copy link
Member

y0urself commented Jul 9, 2024

seems to be, that the formatting is not 100% correct, please fix that. :)

@ITSEC-DACHSER ITSEC-DACHSER marked this pull request as draft July 12, 2024 06:07
auto-merge was automatically disabled July 12, 2024 06:07

Pull request was converted to draft

Added an optional argument to the function create_task to adjust the name
fixed a formatting error in scan-new-system.gmp.py, which caused the build to fail due to code reformatting
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