Open umitdincel opened 3 years ago
Merging #44 (e4654df) into master (36f1242) will decrease coverage by
0.16%
. The diff coverage is66.66%
.
@@ Coverage Diff @@
## master #44 +/- ##
==========================================
- Coverage 71.73% 71.57% -0.17%
==========================================
Files 5 5
Lines 368 380 +12
==========================================
+ Hits 264 272 +8
- Misses 104 108 +4
Impacted Files | Coverage Δ | |
---|---|---|
pyresparser/utils.py | 64.46% <42.85%> (-0.65%) |
:arrow_down: |
pyresparser/resume_parser.py | 76.19% <100.00%> (+0.58%) |
:arrow_up: |
test_name.py | 94.73% <100.00%> (+0.45%) |
:arrow_up: |
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 36f1242...e4654df. Read the comment docs.
@umitdincel it would be great if you add tests for this
@OmkarPathak Added but it doesn't match your linkedin caption so is it possible to update it as full link?
@umitdincel it seems you have not tested the test that you wrote, go through the logs here: https://travis-ci.com/github/OmkarPathak/pyresparser/builds/211156947
The build passed but the coverage looks like a little bit reduced.
It may be useful to add linkedin url to extracting information