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

Modified is_late to check is_billable #828

Merged
merged 1 commit into from
Oct 1, 2018
Merged

Conversation

tbaxter-18f
Copy link
Contributor

Description

Modifies is_late to check for is_billable

Additional information

Also includes some test tidying.

@codecov-io
Copy link

Codecov Report

Merging #828 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #828      +/-   ##
=========================================
+ Coverage   90.89%   90.9%   +0.01%     
=========================================
  Files          39      39              
  Lines        1747    1749       +2     
=========================================
+ Hits         1588    1590       +2     
  Misses        159     159
Impacted Files Coverage Δ
employees/models.py 98.59% <0%> (+0.04%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4a3f65...1946156. Read the comment docs.

Copy link
Contributor

@rogeruiz rogeruiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: 🕔

@rogeruiz rogeruiz merged commit dbb607d into master Oct 1, 2018
@rogeruiz rogeruiz deleted the 827-billable-is-late-check branch October 1, 2018 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants