add .run_line_wrap helper function to wrap long CMD lines in dockerfiles#128
add .run_line_wrap helper function to wrap long CMD lines in dockerfiles#128chainsawriot merged 2 commits intogesistsa:v0.3from
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## v0.3 #128 +/- ##
=======================================
Coverage 97.26% 97.27%
=======================================
Files 10 10
Lines 989 991 +2
=======================================
+ Hits 962 964 +2
Misses 27 27
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
@e-kotov Thanks a lot for the PR! I have provide the CR. Please let me know if you need help with the requested changes. |
|
@chainsawriot I incorporated your review suggestions and written a test. I could not come up with the one that would use |
|
@e-kotov Thanks a lot and I will add you as a CTB. |
Using writeLines / readLines ref #128
Currently the
dockerize()function produces rather unfriendly long lines of RUN commands. For example:With the addition of the new internal (non-exported) helper function
.run_line_wrap()this becomes a more conventional and human readable: