fix: replace File.separator with forward-slash

[paullatzelsperger]

What this PR changes/adds

Replaces the File.separator with a hard-coded forward slash, to avoid path errors on Windows.

Why it does that

Briefly state why the change was necessary.

Further notes

• no tests - Windows is not a supported platform.

Linked Issue(s)

Closes #4219

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

[paullatzelsperger]

@stefan1207 FYI

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.35%. Comparing base (7f20ba5) to head (a2757d5).
Report is 280 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4221      +/-   ##
==========================================
+ Coverage   71.74%   75.35%   +3.61%     
==========================================
  Files         919     1039     +120     
  Lines       18457    20636    +2179     
  Branches     1037     1152     +115     
==========================================
+ Hits        13242    15551    +2309     
+ Misses       4756     4574     -182     
- Partials      459      511      +52     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ndr-brt]

I'm confused, shouldn't be \ the working separator on windows (that should be returned by File.separator on win systems)?

[paullatzelsperger]

I'm confused, shouldn't be \ the working separator on windows (that should be returned by File.separator on win systems)?

no, because it's not a path on the actual file system, but a relative path inside the jar file

[ndr-brt]

amazed by how win can still surprise me after these years! :D