-
Notifications
You must be signed in to change notification settings - Fork 203
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
Print count of rules in the scan wizard #1206
Comments
@YamatoSecurity Thanks for your issue.
I think better 2 than 1. What do you think? |
@hitenkoku Thanks for the question! Yes, I think we should prioritize using little memory and it does not take long to load rules especially if the first time just loads things like |
One thing that might be tricky with counting the number of rules to include is that they will change depending on what the user selects. For example, if there are rules that are both |
@YamatoSecurity Thanks for comment. I will implement option 2.
|
Yes, it will be too complicated to try to dynamically count them and probably confuse users. |
Before:
After:
The text was updated successfully, but these errors were encountered: