pytest-docker-tools icon indicating copy to clipboard operation
pytest-docker-tools copied to clipboard

Make docstring escape sequence proof

Open maltevesper opened this issue 3 years ago • 2 comments

Matters on Windows, where the path can contain backslashes, i.e. C:\User... -> unknown unicode escape sequence.

image

Simply making the docstring in the template a raw string avoids the issue.

maltevesper avatar Jun 08 '22 00:06 maltevesper

Codecov Report

Merging #43 (4ae2274) into main (5853f9b) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #43   +/-   ##
=======================================
  Coverage   93.39%   93.39%           
=======================================
  Files          18       18           
  Lines         560      560           
=======================================
  Hits          523      523           
  Misses         37       37           
Flag Coverage Δ
unittests 93.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pytest_docker_tools/builder.py 100.00% <ø> (ø)

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 5853f9b...4ae2274. Read the comment docs.

codecov[bot] avatar Jun 08 '22 05:06 codecov[bot]

@Jc2k : What do I need to do to get this (tiny, but vital) fix merged?

maltevesper avatar Jul 20 '22 20:07 maltevesper