keeping logs cleaner in dedicated directory

[TheChymera]

Very minor fix, prevents default logs from unceremoniously cluttering /var/tmp/.

[codecov]

Codecov Report

Merging #971 (9a2d233) into master (2c4dcdd) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 9a2d233 differs from pull request most recent head 6b51a8b. Consider uploading reports for the commit 6b51a8b to get more accurate results

@@            Coverage Diff             @@
##           master     #971      +/-   ##
==========================================
+ Coverage   87.52%   87.55%   +0.02%     
==========================================
  Files          64       64              
  Lines        8104     8129      +25     
==========================================
+ Hits         7093     7117      +24     
- Misses       1011     1012       +1     

Flag	Coverage Δ
unittests	87.55% <100.00%> (+0.02%)	⬆️

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

Impacted Files	Coverage Δ
dandi/bids_validator_xs.py	93.00% <100.00%> (-0.16%)	⬇️
dandi/tests/test_bids_validator_xs.py	100.00% <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 2c4dcdd...6b51a8b. Read the comment docs.

[yarikoptic]

replace OS specific paths

[yarikoptic]

if to be fancy let's be fancy consistently with the logs logging ;)

[TheChymera]

@yarikoptic watiting for tests. All fixes suggested seem to be in, except /.

As per https://ppb.chymera.eu/d69110.txt (the report_path variable value is returned in between repeated prints of the strings “lulu” and “lala”) I think os.path somehow already fixes this up, though I am not sure where since I don't have windows locally.

Given that this still auto-works, I think we might leave it as / so as to not clutter the string with additional variables for formatting. Alternatively, if you want to make it extra-safe, I can explicitly replace / with os.sep... tough again, that won't actually be doing anything.

[yarikoptic]

didn't test on windows if would be storing ok given a mix of \ and / in the path, but let's assume so and proceed

[github-actions]

🚀 PR was released in 0.39.3 🚀