valohai / valohai-cli

:heavy_dollar_sign: Command line client for Valohai
MIT License
14 stars 8 forks source link

add linebreak before adding valohai-utils to requirements.txt #240

Closed drazendee closed 2 years ago

drazendee commented 2 years ago

Quickfix.. if a customer has a requirements.txt that doesn't end with an empty line we'll append valohai-utils to the end of the last line, instead of creating a new one

torch==1.13.0
torchvision==0.14.0

There's probably a smorter way to do this :shrug:

codecov[bot] commented 2 years ago

Codecov Report

Merging #240 (0289659) into master (c82cde6) will decrease coverage by 0.04%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #240      +/-   ##
==========================================
- Coverage   86.30%   86.26%   -0.05%     
==========================================
  Files         123      123              
  Lines        4052     4054       +2     
  Branches      664      665       +1     
==========================================
  Hits         3497     3497              
- Misses        385      386       +1     
- Partials      170      171       +1     
Impacted Files Coverage Δ
valohai_cli/commands/yaml/step.py 85.91% <0.00%> (-2.50%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more