search

multitheftauto / mtasa-resources

This project maintains a list of up-to-date resources that come with Multi Theft Auto.
https://multitheftauto.com
MIT License
151 stars 151 forks source link

Lua linter implementation #354

Closed Disinterpreter closed 2 years ago

Disinterpreter commented 2 years ago

Just fast Q&A to my PR

What is it? - https://en.wikipedia.org/wiki/Lint_(software) Why is it need for us? - This repo is a library of examples for newbies and other scripters I think it must be pure and clear for all, even for our GC. Is it need to merge now? - Not necessary. But if you want to see red crossmark on each commit about fixing the problems, you can. Anyway, you can set it as a "Draft". How it looks? - https://github.com/Disinterpreter/mtasa-resources/runs/3697351842 Why are you ignore lot of marks in config? - Linter right now uses for common lua, and it hasn't our functions/variables/tables. We can fork it and change later.

Linter: https://github.com/mpeterv/luacheck

0xCiBeR commented 2 years ago

Nice

Dutchman101 commented 2 years ago

Let's trial it for a month and see how this works out in practise (for PR's & MTA's Lua stack)..