Added a function to add backslash before a space

[MichalKesl]

If path has spaces the function will modify the name and add backslash before the space. a local test was preformed, the solution works.

fixes #535

[rwest]

If you do want to just replace all the with \ then you could consider folder_name.replace(' ','\ ') instead of your split and recombine routine.

In [3]: folder_name = 'foo bar  baz'

In [4]: folder_name.replace(' ','\ ')
Out[4]: 'foo\\ bar\\ \\ baz'

But there may be a better fix. This won't escape anything except spaces.

Most functions don't need the path escaping. The few that do include os.system(), but it might be safer to replace those calls with subprocess and then you avoid the need. But maybe it's for doing something over SSH where shell escaping is not avoidable. In that case, consider shlex.quote()

Return a shell-escaped version of the string s. The returned value is a string that can safely be used as one token in a shell command line, for cases where you cannot use a list.

[alongd]

Thanks, please see some comments

[alongd]

All the modifications to the file up to this point are style changes. If you think they are necessary, please add them as a different commit. I would recommend removing them, the previous version is "OK", I think

[alongd]

this line has no effect (we don't store the returned value)

[alongd]

minor: I would recommend changing i to a different name, i is usually an integer iterator. maybe use split?

[alongd]

You can do either folder_name.split() or folder_name.split(" "), they are the same (the first one is shorter or "cleaner")

[alongd]

If you'd like, you can achieve this in a single line instead of the loop. Try:
new_folder_name = '\ '.join(folder_name.split())

[codecov]

Codecov Report

Merging #541 (df9dff7) into main (c440aab) will increase coverage by 0.01%.
Report is 3 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #541      +/-   ##
==========================================
+ Coverage   73.18%   73.20%   +0.01%     
==========================================
  Files          99       99              
  Lines       26522    26534      +12     
  Branches     5546     5546              
==========================================
+ Hits        19409    19423      +14     
+ Misses       5772     5771       -1     
+ Partials     1341     1340       -1     

Flag	Coverage Δ
unittests	73.20% <100.00%> (+0.01%)	⬆️

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

Files Changed	Coverage Δ
arc/common.py	82.60% <100.00%> (+0.33%)	⬆️
arc/common_test.py	99.85% <100.00%> (+<0.01%)	⬆️
arc/main.py	51.03% <100.00%> (+0.07%)	⬆️

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

[alongd]

Thanks @MichalKesl, looks good!
I added some comments, please take a look

[alongd]

why do we have two slashes in the tests?

[MichalKesl]

If we do michal kfir.replace(" ", "\ ") the string changes to michal\\ kfir, because \ is an operator (like \n).

[alongd]

@MichalKesl, when you get a chance, can we finish this PR?

[calvinp0]

@MichalKesl is this PR relevant?