Skip to content

feat: consider remediation in counts #968

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.

Sign up for GitHub

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

Jump to bottom
Open
MR2011 wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat: consider remediation in counts #968

MR2011 wants to merge 5 commits into from

Conversation

@MR2011
Copy link
Collaborator

@MR2011 MR2011 commented Nov 13, 2025

Description

  • Refactored the mv_vulnerabilities file for the new aggregation table structure
  • Added joins to remediation table in count procedures
  • Moved count issue tests into mv_vulnerabilities_test
  • Added tests

Still ToDo:

  • Tests for all count aggregation tables
  • Tests for GetComponentVulnerabilities in components.go
  • Check if we can remove some aggregation tables

@MR2011 MR2011 force-pushed the mr2011/issue-957/count-remediation branch from 62773a6 to 72225a0 Compare November 18, 2025 09:37
@MR2011 MR2011 marked this pull request as ready for review December 8, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drochow drochow Awaiting requested review from drochow drochow is a code owner

@dorneanu dorneanu Awaiting requested review from dorneanu dorneanu is a code owner

@copyonwrite copyonwrite Awaiting requested review from copyonwrite copyonwrite is a code owner

@michalkrzyz michalkrzyz Awaiting requested review from michalkrzyz michalkrzyz is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@MR2011