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

scan duration in log should be displayed in milis #3477

Closed
LeonidWeinberg opened this issue May 29, 2021 · 1 comment · Fixed by #3504
Closed

scan duration in log should be displayed in milis #3477

LeonidWeinberg opened this issue May 29, 2021 · 1 comment · Fixed by #3504
Assignees
@joaoReigota1
Labels
bug Something isn't working
Milestone
CLI/UI

Comments

@LeonidWeinberg
Copy link

Expected Behavior

scan duration displayed in milis

Actual Behavior

scan duration displayed in seconds with fractions

Steps to Reproduce the Problem

  1. scan any project

It is up to user application to parse the time and display in any format he wants.

Specifications

  • Version: 1.3.1
  • Platform: windows
  • Subsystem:
@LeonidWeinberg LeonidWeinberg added the bug Something isn't working label May 29, 2021
@joaoReigota1 joaoReigota1 self-assigned this May 31, 2021
@joaoReigota1
Copy link
Collaborator

Hi @leonidweinbergcx is it ok if this change is only set for the flag --ci? It is easier for a user to read the scan duration in a format other than milliseconds

joaoReigota1 added a commit that referenced this issue Jun 1, 2021
@joaoReigota1
feature(metrics): Metrics default to 'ms' and 'b' for 'ci' flag #3477 #…
d3d0922 
…3476

Signed-off-by: João Reigota <[email protected]>
@joaoReigota1 joaoReigota1 mentioned this issue Jun 1, 2021
Merged
@rogeriopeixotocx rogeriopeixotocx added this to the CLI/UI milestone Jun 1, 2021
rogeriopeixotocx pushed a commit that referenced this issue Jun 1, 2021
@joaoReigota1
feature(metrics): Metrics default to 'ms' and 'b' for 'ci' flag #3477 #…
24191c5 
…3476 (#3504)

Signed-off-by: João Reigota <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joaoReigota1 joaoReigota1

Labels
bug Something isn't working
Projects
None yet
Milestone
CLI/UI
Development

Successfully merging a pull request may close this issue.

feature(metrics): Metrics default to 'ms' and 'b' for 'ci' flag Checkmarx/kics
3 participants
@joaoReigota1 @rogeriopeixotocx @LeonidWeinberg