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

replace --live_analysis option with --live-analysis #1139

Closed
hitenkoku opened this issue Jul 21, 2023 · 0 comments · Fixed by #1140
Closed

replace --live_analysis option with --live-analysis #1139

hitenkoku opened this issue Jul 21, 2023 · 0 comments · Fixed by #1140
Assignees
@hitenkoku
Labels
enhancement New feature or request
Milestone
v2.7.0

Comments

@hitenkoku
Copy link
Collaborator

I checked that all of the times are correct. There is one thing not related to this PR though. -l, --live_analysis uses _ instead of -. Can you change it to -l, --live-analysis?
(I just noticed it also uses _ for csv-timeline and all the other commands as well, but since we always use hyphens it is probably better to change to --live-analysis)

Originally posted by @YamatoSecurity in #1138 (review)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hitenkoku hitenkoku

Labels
enhancement New feature or request
Projects
None yet
Milestone
v2.7.0
Development

Successfully merging a pull request may close this issue.

Renamed --live_analysis option with --live-analysis Yamato-Security/hayabusa
1 participant
@hitenkoku