Skip to content

Conversation

@chfw
Copy link
Member

@chfw chfw commented May 25, 2019

fix #265 file encoding error when a binary file is being copied.

chfw and others added 9 commits March 17, 2019 09:51
Use simple open in copy engine
* 🔨 Ignore error if occurs

Closes #265

* 🔬 Add tests for encoding

* 🔬 Add tests for encoding

* 🔬 regression test binary file copying
…inja2. hence copy engine is hit with such rediculus, useless procedure. now this is confined to jinja2 only
@chfw chfw requested review from ayan-b and jayvdb May 25, 2019 19:58
@codecov-io
Copy link

Codecov Report

Merging #270 into master will decrease coverage by 0.29%.
The diff coverage is 90.41%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #270     +/-   ##
========================================
- Coverage   99.19%   98.9%   -0.3%     
========================================
  Files          58      60      +2     
  Lines        2478    2460     -18     
========================================
- Hits         2458    2433     -25     
- Misses         20      27      +7
Impacted Files Coverage Δ
moban/_version.py 100% <100%> (ø) ⬆️
moban/utils.py 97.43% <100%> (-0.22%) ⬇️
tests/test_copy_encoding.py 100% <100%> (ø)
tests/test_utils.py 91.66% <100%> (-0.74%) ⬇️
moban/copy/__init__.py 100% <100%> (ø) ⬆️
tests/test_copy_engine.py 100% <100%> (ø) ⬆️
moban/jinja2/engine.py 100% <100%> (ø) ⬆️
moban/plugins/template.py 97.85% <100%> (+0.04%) ⬆️
tests/test_regression.py 80% <80%> (ø)
... and 18 more

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 dfe2066...256fe9b. Read the comment docs.

@codecov-io
Copy link

codecov-io commented May 25, 2019

Codecov Report

Merging #270 into master will decrease coverage by 0.26%.
The diff coverage is 90.41%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #270      +/-   ##
==========================================
- Coverage   99.19%   98.93%   -0.27%     
==========================================
  Files          58       60       +2     
  Lines        2478     2528      +50     
==========================================
+ Hits         2458     2501      +43     
- Misses         20       27       +7
Impacted Files Coverage Δ
moban/_version.py 100% <100%> (ø) ⬆️
moban/utils.py 97.43% <100%> (-0.22%) ⬇️
tests/test_copy_encoding.py 100% <100%> (ø)
tests/test_utils.py 92% <100%> (-0.41%) ⬇️
moban/copy/__init__.py 100% <100%> (ø) ⬆️
tests/test_copy_engine.py 100% <100%> (ø) ⬆️
moban/jinja2/engine.py 100% <100%> (ø) ⬆️
moban/plugins/template.py 97.85% <100%> (+0.04%) ⬆️
tests/test_regression.py 80% <80%> (ø)
... and 1 more

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 dfe2066...256fe9b. Read the comment docs.

@chfw chfw merged commit 818172f into master May 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants