leafo / pgmoon

A pure Lua Postgres driver for use in OpenResty & more
MIT License
390 stars 93 forks source link

travis updates #99

Closed jprjr closed 3 years ago

jprjr commented 3 years ago

Split out from #97 , updates for Travis

leafo commented 3 years ago

Thanks for the contributions. I'm thinking about this and going forward I would prefer to migrate to the lua github actions for testing the lua variants, and migrate away from travis entirely:

https://github.com/leafo/gh-actions-lua https://github.com/leafo/gh-actions-luarocks

postgres could be loaded in with another action for integration testing

In the meantime though I think if it works then we can merge it in, but I don't want to invest too much energy on the travis scripts

jprjr commented 3 years ago

That makes sense, yeah. It should work, I think the current build failed just because #98 wasn't merged in yet with the fix for checking os.execute

leafo commented 3 years ago

So it seems like travis is on its way out even faster, so I migrated over to github actions. All the travis code has now been removed from the repo. Most of the issues listed here are resolved with the exception of support for testing on 32bit Lua.

That's going to have to be a patch to the Lua action here to allow build flags to be passed: https://github.com/leafo/gh-actions-lua

Thanks for the patch, sorry I wasn't able to accept it as is, but at least we've now got much better Lua coverage