Skip to content

Conversation

@glarrain
Copy link
Contributor

@glarrain glarrain commented Apr 5, 2019

Code for parsing DTE data from representation as XML document (support for other representations may be added later on).

Main function is parse_dte_xml, and the helpers are clean_dte_xml and validate_dte_xml.

Tests have not been implemented.

Code for parsing DTE data from representation as XML document
(support for other representations may be added later on).

Main function is `parse_dte_xml`, and the helpers are `clean_dte_xml`
and `validate_dte_xml`.

Tests have not been implemented.
@glarrain glarrain self-assigned this Apr 5, 2019
@glarrain glarrain merged commit b3692fa into develop Apr 5, 2019
@codecov-io
Copy link

Codecov Report

Merging #11 into develop will decrease coverage by 4.74%.
The diff coverage is 29.88%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #11      +/-   ##
===========================================
- Coverage    59.77%   55.02%   -4.75%     
===========================================
  Files           13       14       +1     
  Lines          440      527      +87     
  Branches        50       62      +12     
===========================================
+ Hits           263      290      +27     
- Misses         176      235      +59     
- Partials         1        2       +1
Impacted Files Coverage Δ
cl_sii/dte/parse.py 29.88% <29.88%> (ø)
cl_sii/libs/xml_utils.py 53.06% <0%> (+2.04%) ⬆️

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 f58ed33...ceb260e. Read the comment docs.

@glarrain glarrain deleted the feature/dte/add-parse branch April 5, 2019 14:56
@glarrain glarrain mentioned this pull request Apr 5, 2019
@jtrobles-cdd jtrobles-cdd added enhancement New feature or request and removed feature labels Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: dte enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants