Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add cascade CLI Command To Visualize Cascaded Scans Hierarchy #2608

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

Merged
merged 15 commits into from
Aug 18, 2024

Conversation

Freedisch
Copy link
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.

Copy link

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 Freedisch requested a review from J12934 August 15, 2024 21:17
Copy link
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
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 <[email protected]>
Signed-off-by: Freedisch <[email protected]>
Co-authored-by: Jannik Hollenbach <[email protected]>
Signed-off-by: Freedisch <[email protected]>
@Freedisch Freedisch requested a review from J12934 August 16, 2024 13:51
Signed-off-by: freedisch <[email protected]>
Copy link

@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
54 checks passed
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