Skip to content

Conversation

@jayvdb
Copy link
Member

@jayvdb jayvdb commented Jul 9, 2019

No description provided.

jayvdb added 2 commits July 9, 2019 11:04
The Python 3.8-dev job has broken on building ruamel.yaml,
and the template doesnt provide control over allowing failures.
Fixes two UnicodeEncodeError's occurring when hashing
and writing files.

Closes moremoban#281
@codecov-io
Copy link

codecov-io commented Jul 9, 2019

Codecov Report

Merging #287 into dev will decrease coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #287      +/-   ##
==========================================
- Coverage   98.94%   98.91%   -0.03%     
==========================================
  Files          60       60              
  Lines        2550     2487      -63     
==========================================
- Hits         2523     2460      -63     
  Misses         27       27
Impacted Files Coverage Δ
moban/utils.py 97.53% <100%> (+0.09%) ⬆️
moban/hashstore.py 100% <100%> (ø) ⬆️
tests/test_utils.py 91.66% <0%> (-0.34%) ⬇️
tests/mobanfile/test_targets.py 97.05% <0%> (-0.05%) ⬇️
moban/mobanfile/__init__.py 99.24% <0%> (-0.01%) ⬇️
tests/mobanfile/test_mobanfile.py 100% <0%> (ø) ⬆️
tests/test_engine.py 100% <0%> (ø) ⬆️
tests/test_main.py 100% <0%> (ø) ⬆️
moban/jinja2/filters/repr.py 100% <0%> (ø) ⬆️
moban/data_loaders/json_loader.py 100% <0%> (ø) ⬆️
... and 9 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 e6b131f...10bebef. Read the comment docs.

@chfw
Copy link
Member

chfw commented Jul 10, 2019

Do you have the artifact?

@jayvdb
Copy link
Member Author

jayvdb commented Jul 10, 2019

Sorry .. which artifact ? The error? it is included in the issue which is mentioned in the commit message : #281

Copy link
Member

@chfw chfw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@chfw chfw merged commit 7dc6379 into moremoban:dev Jul 11, 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.

3 participants