Skip to content

Conversation

@glarrain
Copy link
Contributor

@glarrain glarrain commented May 8, 2019

Rename to signature_x509_cert_der and hold the X.509 certificate's DER-encoded data instead of PEM-encoded data.

Breaks backwards compatibility.

Rename to `signature_x509_cert_der` and hold the X.509 certificate's
DER-encoded data instead of PEM-encoded data.

**Breaks backwards compatibility**.
@glarrain glarrain added enhancement New feature or request component: dte labels May 8, 2019
@glarrain glarrain requested a review from jtrh May 8, 2019 16:41
@glarrain glarrain self-assigned this May 8, 2019
@codecov-io
Copy link

Codecov Report

Merging #38 into develop will decrease coverage by 0.46%.
The diff coverage is 57.14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop     #38      +/-   ##
==========================================
- Coverage    72.17%   71.7%   -0.47%     
==========================================
  Files           16      16              
  Lines          769     767       -2     
  Branches       101     101              
==========================================
- Hits           555     550       -5     
- Misses         173     174       +1     
- Partials        41      43       +2
Impacted Files Coverage Δ
cl_sii/dte/parse.py 85.18% <100%> (ø) ⬆️
cl_sii/dte/data_models.py 57.63% <50%> (-1.96%) ⬇️
cl_sii/libs/encoding_utils.py 78.26% <0%> (-4.35%) ⬇️

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 028589e...99ca4da. Read the comment docs.

@glarrain glarrain merged commit a1919ff into develop May 8, 2019
@glarrain glarrain deleted the feature/dte/data_models/alter-field-2 branch May 8, 2019 16:43
@glarrain glarrain mentioned this pull request May 8, 2019
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.

3 participants