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.

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

"jf docker scan" panic fix #1291

Merged
merged 5 commits into from
Nov 4, 2024
Merged

"jf docker scan" panic fix #1291

merged 5 commits into from
Nov 4, 2024

Conversation

dortam888
Copy link
Contributor

@dortam888 dortam888 commented Nov 3, 2024 •
edited
Loading

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.

Fix an issue with jf dockerscan that had panic at usage.
Additionally, Fixed a typo in the usage that show jf jf docker scan instead of jf docker scan

@eyalbe4 eyalbe4 changed the title Dockerscan panic fix "jf docker scan" panic fix Nov 4, 2024
@eyalbe4 eyalbe4 added the bug Something isn't working label Nov 4, 2024
@eyalbe4 eyalbe4 merged commit 3029fdc into jfrog:dev Nov 4, 2024
6 of 7 checks passed
@BrewTestBot BrewTestBot mentioned this pull request Nov 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@attiasas attiasas Awaiting requested review from attiasas

@eyalbe4 eyalbe4 Awaiting requested review from eyalbe4

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dortam888 @eyalbe4 @attiasas