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.

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

Add colors to the vulnerability scan module #9

Merged
merged 3 commits into from
Feb 1, 2022
Merged

Conversation

g3rzi
Copy link

@g3rzi g3rzi commented Jan 31, 2022

Pull Request (PR) Checklist

  • I have read the CONTRIBUTING doc
  • PR is from a topic/feature/bugfix branch off the dev branch (right side)
  • PR is against the dev branch (left side)
  • Merlin compiles without errors
  • Passes linting checks and unit tests
  • Updated CHANGELOG
  • Updated README documentation (if applicable)
  • Update Merlin version number in pkg/merlin.go (if applicable)

Change Type

  • Addition
  • Bugfix
  • Modification
  • Removal
  • Security

Description

I added colors to make the results more clearly for the user.
It now looks like that:
Before:
image

After:
image

Not sure if the log's colors are too much, we can also think to make the logs simpler but it also OK to leave it like that.
I think that we can do the same changes in other modules to make easier to read for the user.

@g3rzi g3rzi merged commit a6ee769 into cyberark:main Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant