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

Fix command duration issues, also add show subsecond support #213

Merged

Conversation

novadev94
Copy link
Contributor

@novadev94 novadev94 commented Mar 25, 2020

  • Fix command duration rounding: Duration wasn't shown when:
    • When threshold = 5 (5000ms)
    • When process finished in [5000, 6000) miliseconds
  • Fix command duration threshold: Duration wasn't shown when:
    • When threshold = 5 (5000ms)
    • When process finished in 1m4s (64 seconds)
  • Add support for showing subsecond in command duration
    via pure_show_subsecond_command_duration

@novadev94 novadev94 changed the title Fix command duration issues, also add show miliseconds support Fix command duration issues, also add show subsecond support Mar 26, 2020
@novadev94 novadev94 force-pushed the fix/command_duration_rounding branch from c13e5c2 to de39f76 Compare March 26, 2020 22:48
@edouard-lopez edouard-lopez added 🐛 bug something that doesn't works as expected 🚀 enhancement performance, UX or maintainability labels Dec 1, 2020
novadev94 and others added 9 commits December 11, 2020 10:01
- Fix command duration rounding: Duration wasn't shown when:
    + When threshold = 5 (5000ms)
    + When process finished in [5000, 6000) miliseconds
- Fix command duration threshold: Duration wasn't shown when:
    + When threshold = 5 (5000ms)
    + When process finished in 1m4s (64 seconds)
- Add support for showing subsecond in command duration
  via `pure_show_subsecond_command_duration`
@edouard-lopez edouard-lopez merged commit 48a5a5b into pure-fish:master Dec 11, 2020
@novadev94 novadev94 deleted the fix/command_duration_rounding branch December 14, 2020 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug something that doesn't works as expected 🚀 enhancement performance, UX or maintainability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants