Skip to content

Conversation

enriquezrene
Copy link
Contributor

  • Removes modal component
  • Removes flaky test on update from time-clock screen
  • Add missing tests for project actions

@codecov
Copy link

codecov bot commented May 19, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #267      +/-   ##
==========================================
+ Coverage   93.63%   94.64%   +1.00%     
==========================================
  Files          63       62       -1     
  Lines         959      953       -6     
  Branches       57       57              
==========================================
+ Hits          898      902       +4     
+ Misses         42       32      -10     
  Partials       19       19              
Impacted Files Coverage Δ
...nents/projects/components/store/project.actions.ts 100.00% <0.00%> (+8.00%) ⬆️
...modules/shared/components/clock/clock.component.ts 91.66% <0.00%> (+33.33%) ⬆️

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 3c7e789...44deee7. Read the comment docs.

@enriquezrene enriquezrene merged commit 705c72b into master May 19, 2020
@enriquezrene enriquezrene deleted the 265-remove-modal-component branch May 19, 2020 18:24
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.

1 participant