teal-language / cyan

The Teal build system and project manager
MIT License
62 stars 6 forks source link

Ensure tests run on windows #9

Open euclidianAce opened 3 years ago

euclidianAce commented 3 years ago

Additionally, find some way to run CI for windows as leafo's github actions do not support it. So either fix that upstream or find a different method.

euclidianAce commented 3 years ago

I have made a futile attempt to patch leafo's github action to support windows. It fetches and builds Lua perfectly fine, the issue seems to be with adding it to the path. I have no idea what the trouble is, luajit seems to work fine but it makes a symlink somewhere instead/in addition to adding to PATH. So I have no idea

euclidianAce commented 3 years ago

I think I'll push this back to the 0.2 milestone as I've used cyan on windows a little bit and it seems to work for the most part. I'd like to focus on a little bit of polish for the first release. CI can be worried about after that.

If anybody knows of a good github action to use, I'm all ears. Otherwise my tentative plan is to maybe write a custom action for hererocks

hishamhm commented 2 years ago

I'm also curious about this! I have just converted the LuaRocks CI from Travis CI to Github Actions — just for Linux and Mac, though; Windows still runs on Appveyor. So If anyone finds a way to run Github Actions for Lua stuff on Windows cleanly, I'm also interested! Tackling this is not a priority for me now since Appveyor is working fine, but it would be nice to simplify things.

hishamhm commented 2 years ago

@euclidianAce Keep an eye on https://github.com/leafo/gh-actions-luarocks/pull/14 :)