Add windows_file.template operation #651
Conversation
Codecov Report
@@ Coverage Diff @@ ## current pyinfra-dev/pyinfra#651 +/- ## =========================================== - Coverage 92.51% 91.69% -0.83% =========================================== Files 123 121 -2 Lines 7440 7367 -73 =========================================== - Hits 6883 6755 -128 - Misses 557 612 +55
Continue to review full report at Codecov.
|
Fizzadar left a comment
Fizzadar left a comment There was a problem hiding this comment.
Hi @timeisapear - thank you for adding this, looks great overall. Once minor comment and the tests - I'm happy to add tests after merging for this if you'd like me to?
| .. code:: python | ||
| files.template( |
There was a problem hiding this comment.
These examples need to be windows_files.templates.
Fizzadar force-pushed the next branch 2 times, most recently from 5a9e175 to f01ba12 Compare 
Mostly copy/pasted from file.template. One tweak is making the IO buffer-type bytes since win_rm _put_file expects a bytes input. Therefore output_file was changed from: ``` output_file = six.StringIO(output) ``` to ``` output_file = six.BytesIO(six.ensure_binary(output)) ``` References https://github.com/Fizzadar/pyinfra/issues/645
Fizzadar force-pushed the add-windows_file-template branch from be8f856 to 325b214 Compare 
| v3 will remove Windows support (into a separate package, https://github.com/pyinfra-dev/pyinfra-windows. Once that's released I shall move this over and close this PR. |
| This PR also seems to be dead. |
3 participants
Mostly copy/pasted from file.template. One tweak is making the IO buffer-type bytes since
winrm connector _put_file expects a bytes input. Therefore output_file was changed from:
to
References https://github.com/Fizzadar/pyinfra/issues/645
I have not added tests yet but may be able to add them later.