Skip to content

Add/get report exports - #1375

Merged
greenbonebot merged 5 commits into
mainfrom
add/get-report-exports
Sep 4, 2026
Merged

Add/get report exports#1375
greenbonebot merged 5 commits into
mainfrom
add/get-report-exports

Conversation

@ozgen

@ozgen ozgen commented Sep 4, 2026

Copy link
Copy Markdown
Member

What

  • Add get_report_exports request support
  • Add tests for request generation

Why

  • Allow python-gvm clients to use the new get_report_exports GMP command

References

GEA-2042

Checklist

  • Tests

@ozgen
ozgen requested a review from a team as a code owner September 4, 2026 06:23
@greenbonebot
greenbonebot enabled auto-merge (rebase) September 4, 2026 06:23
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA b71fb2a.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Conventional Commits Report

Type Number
Changed 2
Added 2

🚀 Conventional commits found.

@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.37%. Comparing base (201a2bd) to head (b71fb2a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1375   +/-   ##
=======================================
  Coverage   98.37%   98.37%           
=======================================
  Files         103      104    +1     
  Lines        6767     6787   +20     
  Branches     1230     1231    +1     
=======================================
+ Hits         6657     6677   +20     
  Misses         77       77           
  Partials       33       33           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@timopollmeier timopollmeier 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.

In python_gvm we usually separate get_... commands into a singular one with an id parameter and a plural one for the list.

@ozgen
ozgen requested a review from timopollmeier September 4, 2026 06:54
@ozgen

ozgen commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

In python_gvm we usually separate get_... commands into a singular one with an id parameter and a plural one for the list.

fixed with 8ff43ad

Comment thread gvm/protocols/gmp/requests/next/_report_exports.py Outdated
@ozgen
ozgen force-pushed the add/get-report-exports branch from 8ff43ad to fa824d3 Compare September 4, 2026 07:04
@ozgen
ozgen force-pushed the add/get-report-exports branch from fa824d3 to 4749a4b Compare September 4, 2026 07:07
@ozgen
ozgen requested a review from timopollmeier September 4, 2026 07:08
@ozgen
ozgen force-pushed the add/get-report-exports branch from 8b1467f to b71fb2a Compare September 4, 2026 07:21
@greenbonebot
greenbonebot merged commit 2b42a13 into main Sep 4, 2026
26 checks passed
@greenbonebot
greenbonebot deleted the add/get-report-exports branch September 4, 2026 07:47
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