Skip to content

Conversation

@glarrain
Copy link
Contributor

Closes #43.

@glarrain glarrain requested a review from a team May 15, 2019 16:41
@glarrain glarrain self-assigned this May 15, 2019
@codecov-io
Copy link

Codecov Report

Merging #44 into develop will decrease coverage by 0.37%.
The diff coverage is 67.74%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #44      +/-   ##
===========================================
- Coverage    71.78%   71.41%   -0.38%     
===========================================
  Files           16       16              
  Lines          762      780      +18     
  Branches       100      106       +6     
===========================================
+ Hits           547      557      +10     
- Misses         173      177       +4     
- Partials        42       46       +4
Impacted Files Coverage Δ
cl_sii/dte/parse.py 80.15% <67.74%> (-5.03%) ⬇️
cl_sii/dte/data_models.py 58.27% <0%> (+0.71%) ⬆️

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 31c1972...75f681b. Read the comment docs.

@glarrain glarrain merged commit 1bf6fa0 into develop May 15, 2019
@glarrain glarrain deleted the bugfix/dte/parse/fix-parse_dte_xml branch May 15, 2019 16:43
@glarrain glarrain mentioned this pull request May 15, 2019
@jtrobles-cdd jtrobles-cdd added bug Something isn't working and removed bugfix labels Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component: dte

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dte.parse: missing "emisor" or "receptor" email value breaks parse_dte_xml

4 participants