-
Notifications
You must be signed in to change notification settings - Fork 205
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
feat: add --disable-abbreviations
option
#1497
Conversation
csv-timelinehelp
--disable-abbreviations option
|
json-timelinehelp
--disable-abbreviations option
|
eid-metricshelp
--disable-abbreviations option
|
log-metricshelp
-disable-abbreviations option
|
searchhelp
-disable-abbreviations option
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fukusuket LGTM! Just one thing, I think this option will be better placed under Output
instead of General Options
as that is where other similar options are. Could you just move this?
@YamatoSecurity |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fukusuket LGTM! Thanks so much!
What Changed
--disable-abbreviations
option #1485-a
option was already in use, I temporarily set it to the-b
option.The command to which this option is added
Evidence
Integration-Test
I would appreciate it if you could check it out when you have time🙏