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

refactor: refactoring TimeEstimates class #154

Merged

vvatanabe merged 1 commit into main from refactoring-TimeEstimates

Aug 24, 2023

Member

vvatanabe commented Aug 24, 2023

in TimeEstimates:

define reused values as constants
streamline displayTime conditions
simplify format method


          refactor: define reused values as constants, streamline displayTime c…

01ce7c0

…onditions, and simplify format method in TimeEstimates

coveralls commented Aug 24, 2023

coverage: 92.672% (+0.009%) from 92.663% when pulling 01ce7c0 on refactoring-TimeEstimates into 5cc26d3 on main.

vvatanabe merged commit 2cc6230 into main

3 checks passed

vvatanabe deleted the refactoring-TimeEstimates branch

August 25, 2023 11:55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment