sailorproject / sailor

A Lua MVC Web Framework.
MIT License
921 stars 125 forks source link

Update Lua version in CONTRIBUTING.md #101

Closed luisbg closed 8 years ago

luisbg commented 8 years ago

To keep consistency with the version recommended in docs/INSTALL_LINUX.md

Etiene commented 8 years ago

Hm... I'm wondering if we should set the linux docs to 5.1 instead. Although sailor is compatible with both versions of Lua, depending on the webserver used, such as nginx, it will require 5.1...

luisbg commented 8 years ago

I could offer a patch where docs/INSTALL_LINUX.md suggests the lua5.1 debian package instead. The main issue I am trying to correct is the inconsitency, which I noticed because I am starting to play with the project with the intent of doing bigger things later on :)

Thanks for your very prompt reply

Etiene commented 8 years ago

Yea, maybe that would be better :)

Etiene commented 8 years ago

Thanks for catching this! :D