Skip to content

Conversation

jr-98
Copy link
Contributor

@jr-98 jr-98 commented Dec 16, 2020

This PR resolve some vulnerabilities

@codecov
Copy link

codecov bot commented Dec 16, 2020

Codecov Report

Merging #590 (655e788) into master (13de42e) will increase coverage by 0.33%.
The diff coverage is 98.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #590      +/-   ##
==========================================
+ Coverage   93.09%   93.42%   +0.33%     
==========================================
  Files          85       95      +10     
  Lines        1564     1674     +110     
  Branches      107      109       +2     
==========================================
+ Hits         1456     1564     +108     
- Misses         67       72       +5     
+ Partials       41       38       -3     
Impacted Files Coverage Δ
src/app/app-routing.module.ts 100.00% <ø> (ø)
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...dules/time-entries/pages/time-entries.component.ts 83.33% <85.71%> (+0.52%) ⬆️
src/app/modules/users/store/user.reducers.ts 87.50% <87.50%> (ø)
src/app/guards/login-guard/login.guard.ts 100.00% <100.00%> (ø)
...nologies-report-guard/technologies-report.guard.ts 100.00% <100.00%> (ø)
src/app/modules/login/login.component.ts 100.00% <100.00%> (+16.66%) ⬆️
...app/modules/login/services/azure.ad.b2c.service.ts 86.95% <100.00%> (+4.60%) ⬆️
...ponents/details-fields/details-fields.component.ts 89.61% <100.00%> (+2.43%) ⬆️
...les/shared/components/sidebar/sidebar.component.ts 92.00% <100.00%> (+3.11%) ⬆️
... and 28 more

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 0cb1516...655e788. Read the comment docs.

@josepato87 josepato87 merged commit cdb0365 into master Dec 18, 2020
@josepato87 josepato87 deleted the TT-80 branch December 18, 2020 15:22
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.

6 participants