feat: add github action pr builds for building project, and running tests on linux and windows

[wmundev]

FYI @twcclegg , these are PR builds to run the unit tests on github actions whenever someone makes a new PR if you are interested in it

github actions are failing because of the zip files delimiter now only supports \ and not /

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f6c955e) 81.96% compared to head (7d1dccd) 82.65%.
Report is 12 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #204      +/-   ##
==========================================
+ Coverage   81.96%   82.65%   +0.68%     
==========================================
  Files          30       30              
  Lines        3494     3499       +5     
  Branches      975      978       +3     
==========================================
+ Hits         2864     2892      +28     
+ Misses        320      298      -22     
+ Partials      310      309       -1     

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

[twcclegg]

Hey there, thanks for all your contributions.
Per the path separator, that seems to have caused some problem #212
Eventually I would like to fully move all of the CI from AppVeyor to GHA; one consideration being, if it's deemed okay to drop running tests on netframework or if it's worth the added complexity of having to add a Windows build step.

[twcclegg]

Given all of your work, any interest in becoming a more official collaborator?

[wmundev]

@twcclegg thanks for the link to the issue, i've been trying to replicate the IndexOutOfRangeException issue but to no avail with the code change i made around the path separator, something that might be close that i found is this page here

however it doesn't make sense to me as the code change we did should have supported both path separators

yeah sure happy to be a official collaborator 👍

[wmundev]

@twcclegg what do you think about this implementation of a safe fallback if the original implementation of the directory char doesn't work? - 7d1dccd

this will resolve the pr builds failing but also ensure original implementation works as expected