Skip to content

Add cascade CLI Command To Visualize Cascaded Scans Hierarchy#2608

Merged
J12934 merged 15 commits intosecureCodeBox:mainfrom
Freedisch:cascading_rule
Aug 18, 2024
Merged

Add cascade CLI Command To Visualize Cascaded Scans Hierarchy#2608
J12934 merged 15 commits intosecureCodeBox:mainfrom
Freedisch:cascading_rule

Conversation

@Freedisch
Copy link
Copy Markdown
Member

@Freedisch Freedisch commented Aug 2, 2024

Description

This PR adds a new cascade command to display scans like a tree
stream issue #189

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 2, 2024

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit 284dbe3
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/66c0476eb6f5fe000897fffe

@J12934 J12934 added the cli Issues regarding the CLI Integration label Aug 2, 2024
@Freedisch Freedisch self-assigned this Aug 2, 2024
Freedisch and others added 8 commits August 6, 2024 22:12
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
@Freedisch Freedisch requested a review from J12934 August 15, 2024 21:17
Copy link
Copy Markdown
Member

@J12934 J12934 left a comment

Choose a reason for hiding this comment

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

Awesome, just tested this with a namespace with a couple of hundreds of scans and it works really well 🙌

Some smaller code style comments but other than that it should be good to go 🚀

return m.Client, m.namespace, m.err
}

type testcases struct {
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.

Also seems unused, the tests seem to be using a inline type

Freedisch and others added 3 commits August 16, 2024 14:10
Co-authored-by: Jannik Hollenbach <jannik@hollenbach.de>
Signed-off-by: Freedisch <82499435+Freedisch@users.noreply.github.com>
Co-authored-by: Jannik Hollenbach <jannik@hollenbach.de>
Signed-off-by: Freedisch <82499435+Freedisch@users.noreply.github.com>
Signed-off-by: freedisch <freeproduc@gmail.com>
@Freedisch Freedisch requested a review from J12934 August 16, 2024 13:51
Signed-off-by: freedisch <freeproduc@gmail.com>
@sonarqubecloud
Copy link
Copy Markdown

@J12934 J12934 added the enhancement New feature or request label Aug 18, 2024
@J12934 J12934 changed the title [Feature] Add cascade command for Cascading rules Add cascade CLI Command To Visualize Cascaded Scans Hierarchy Aug 18, 2024
@J12934 J12934 merged commit fba73e7 into secureCodeBox:main Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli Issues regarding the CLI Integration enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants