pyresparser icon indicating copy to clipboard operation
pyresparser copied to clipboard

Support linkedin url for extracting information

Open umitdincel opened this issue 5 years ago • 5 comments

It may be useful to add linkedin url to extracting information

umitdincel avatar Dec 18 '20 22:12 umitdincel

Codecov Report

Merging #44 (e4654df) into master (36f1242) will decrease coverage by 0.16%. The diff coverage is 66.66%.

Impacted file tree graph

@@            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.

codecov[bot] avatar Dec 18 '20 22:12 codecov[bot]

@umitdincel it would be great if you add tests for this

OmkarPathak avatar Dec 26 '20 16:12 OmkarPathak

@OmkarPathak Added but it doesn't match your linkedin caption so is it possible to update it as full link?

umitdincel avatar Dec 29 '20 11:12 umitdincel

@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

OmkarPathak avatar Jan 02 '21 16:01 OmkarPathak

The build passed but the coverage looks like a little bit reduced.

umitdincel avatar Jan 08 '21 22:01 umitdincel