ci: add executors for macOS and Windows

nodejs / postject

Easily inject arbitrary read-only resources into executable formats (Mach-O, PE, ELF) and use it at runtime.

Other

188 stars 14 forks source link

ci: add executors for macOS and Windows #5

Closed dsanders11 closed 2 years ago

dsanders11 commented 2 years ago

Couple of rough edges in this PR but just trying to unblock:

GitHub is showing the Windows job name as default instead of build-win/default
Tests aren't run on Windows currently

dsanders11 commented 2 years ago

is getting run on macOS

Thanks, fixed.

Do you plan to run the tests on Windows before merging this PR?

Not in this PR, so it can be merged in a more timely manner. I don't think the Makefile for tests will be the long term solution so I'm not going to invest time in it for Windows and delay this PR.